<p dir="ltr">I'd have to agree with Martin here.  The output of 2 functions should surely match.  The fact that the referencesource uses something that defaults to something different per platform should be irrelevant.</p>
<p dir="ltr">The only point to consider is, should the output of a method be able to differ per platform and still be valid.  For this situation I imagine that the xaml that is being produced is invalid, and therefore the behaviour for the xaml writer is incorrect.</p>
<div class="gmail_quote">On 29 Sep 2015 05:55, "Atsushi Eno" <<a href="mailto:atsushieno@veritas-vos-liberabit.com">atsushieno@veritas-vos-liberabit.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">You have no reason to be afraid of asking Microsoft to change the<br>
behavior. I cannot find any justification to just ask Mono to change<br>
things against Microsoft code behavior without asking Microsoft about it.<br>
<br>
Atsushi Eno<br>
<br>
On 2015年09月29日 01:15, Martin Potter wrote:<br>
> I wasn’t aware that the MSDN .NET documentation was viewed as being<br>
> platform specific documentation.<br>
><br>
> I am not sure how this is a Microsoft bug so much as an issue with the<br>
> fact they did not consider it being run on other platforms. Given that<br>
> Mono is cross-platform, whereas Microsoft’s implementation is not, and<br>
> Mono has made other changes to the reference source (which is very Windows<br>
> specific) to provide consistency across platforms it is unclear as to why<br>
> the previous behavior of using \r\n (prior to the switch to the reference<br>
> source) is incorrect.<br>
><br>
> ― Martin<br>
><br>
> On 9/28/15, 9:58 AM, "Atsushi Eno" <<a href="mailto:atsushieno@veritas-vos-liberabit.com">atsushieno@veritas-vos-liberabit.com</a>><br>
> wrote:<br>
><br>
>> MSDN documentation describes the behavior how .NET Framework works on<br>
>> Windows, hence it says \r\n. As the referencesource is explicitly<br>
>> implemented, it is NOT the case on non-Windows platform.<br>
>><br>
>> If you are unhappy about Microsoft behavior, you should file a bug to<br>
>> Microsoft. referencesource is not for reporting issues though (as its<br>
>> README explicitly says so).<br>
>><br>
>> Atsushi Eno<br>
>><br>
>> On 2015年09月28日 23:15, Martin Potter wrote:<br>
>>> The default for XamlWriterSettings NewLineChars was changed from "\r\n"<br>
>>> to<br>
>>> "\n" when you imported the reference source and is what is present on<br>
>>> the<br>
>>> 4.2 branch. This is not what the previous behavior was, nor what the<br>
>>> documentation on MSDN,<br>
>>><br>
>>> <a href="https://msdn.microsoft.com/en-us/library/system.xml.xmlwritersettings.new" rel="noreferrer" target="_blank">https://msdn.microsoft.com/en-us/library/system.xml.xmlwritersettings.new</a><br>
>>> li<br>
>>> nechars(v=vs.110).aspx, states.<br>
>>><br>
>>> "The character string to use for line breaks. This can be set to any<br>
>>> string value. However, to ensure valid XML, you should specify only<br>
>>> valid<br>
>>> white space characters, such as space characters, tabs, carriage<br>
>>> returns,<br>
>>> or line feeds. The default is \r\n (carriage return, new line)."<br>
>>><br>
>>> ― Martin<br>
>>><br>
>>> On 9/28/15, 8:08 AM, "Atsushi Eno"<br>
>>> <<a href="mailto:atsushieno@veritas-vos-liberabit.com">atsushieno@veritas-vos-liberabit.com</a>><br>
>>> wrote:<br>
>>><br>
>>>> Current Microsoft-based implementation is the correct one and it's been<br>
>>>> there in post-4.0 versions.<br>
>>>><br>
>>>> Atsushi Eno<br>
>>>><br>
>>>> On 2015年09月28日 22:46, Martin Potter wrote:<br>
>>>>> Atsushi,<br>
>>>>><br>
>>>>> Were you (or someone else) going to get this fixed before 4.2 is ready<br>
>>>>> to<br>
>>>>> ship?<br>
>>>>><br>
>>>>> Thank you,<br>
>>>>> Martin<br>
>>>>><br>
>>>>> On 9/8/15, 8:03 PM, "Atsushi Eno"<br>
>>>>> <<a href="mailto:atsushieno@veritas-vos-liberabit.com">atsushieno@veritas-vos-liberabit.com</a>><br>
>>>>> wrote:<br>
>>>>><br>
>>>>>> I had no idea when that commit was made, but that change was wrong.<br>
>>>>>> Microsoft behavior is the one we should use in that case.<br>
>>>>>><br>
>>>>>> Atsushi Eno<br>
>>>>>><br>
>>>>>> On 2015年09月09日 01:50, Martin Potter wrote:<br>
>>>>>>> Way back in 2012, this commit<br>
>>>>>>><br>
>>>>>>><br>
>>>>>>><br>
>>>>>>> <a href="https://github.com/mono/mono/commit/9215ddf7ade5f0f0f54c2adb7b165e4e3" rel="noreferrer" target="_blank">https://github.com/mono/mono/commit/9215ddf7ade5f0f0f54c2adb7b165e4e3</a><br>
>>>>>>> e7<br>
>>>>>>> 74<br>
>>>>>>> 33b<br>
>>>>>>><br>
>>>>>>><br>
>>>>>>><br>
>>>>>>> <<a href="https://github.com/LogosBible/mono/commit/9215ddf7ade5f0f0f54c2adb7b" rel="noreferrer" target="_blank">https://github.com/LogosBible/mono/commit/9215ddf7ade5f0f0f54c2adb7b</a><br>
>>>>>>> 16<br>
>>>>>>> 5e<br>
>>>>>>> 4e3e77433b> changed<br>
>>>>>>> the default newline in XmlWriterSettings to be "\r\n” to be<br>
>>>>>>> compatible<br>
>>>>>>> with MS (which some of our tests rely on). It appears that when the<br>
>>>>>>> reference source was imported in<br>
>>>>>>><br>
>>>>>>><br>
>>>>>>><br>
>>>>>>> <a href="https://github.com/mono/mono/commit/4990b4c47b8fe1b219c64884fad98d586" rel="noreferrer" target="_blank">https://github.com/mono/mono/commit/4990b4c47b8fe1b219c64884fad98d586</a><br>
>>>>>>> 15<br>
>>>>>>> 46<br>
>>>>>>> a05#diff-5561e6ab8de20ec168dcb1f7b415b353,<br>
>>>>>>> it was changed back to be platform specific. Was this an intentional<br>
>>>>>>> change?<br>
>>>>>>><br>
>>>>>>> Thank you,<br>
>>>>>>> Martin<br>
>>>>>>><br>
>>>>>>><br>
>>>>>>><br>
>>>>>>> _______________________________________________<br>
>>>>>>> Mono-devel-list mailing list<br>
>>>>>>> <a href="mailto:Mono-devel-list@lists.ximian.com">Mono-devel-list@lists.ximian.com</a><br>
>>>>>>> <a href="http://lists.ximian.com/mailman/listinfo/mono-devel-list" rel="noreferrer" target="_blank">http://lists.ximian.com/mailman/listinfo/mono-devel-list</a><br>
>>>>>>><br>
>>>>>><br>
>>>>><br>
>>>>><br>
>>>>><br>
>>>><br>
>>><br>
>>><br>
>>><br>
>><br>
><br>
><br>
><br>
<br>
_______________________________________________<br>
Mono-devel-list mailing list<br>
<a href="mailto:Mono-devel-list@lists.ximian.com">Mono-devel-list@lists.ximian.com</a><br>
<a href="http://lists.ximian.com/mailman/listinfo/mono-devel-list" rel="noreferrer" target="_blank">http://lists.ximian.com/mailman/listinfo/mono-devel-list</a><br>
</blockquote></div>