Hey,<div><br></div><div>I wonder if we'd be better off deleting the actual code from our repository and pulling it directly from the microsoft one via a git submodule. It should be an easier way to keep things up to date and also simpler to maintain patches/modifications (should any be required). <br>
<br>Alan</div><div><br><div class="gmail_quote">On 28 March 2012 13:30, Atsushi Eno <span dir="ltr"><<a href="mailto:atsushieno@veritas-vos-liberabit.com">atsushieno@veritas-vos-liberabit.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hello,<br>
<br>
As many of you guys already know, Microsoft did a huge <a href="http://ASP.NET" target="_blank">ASP.NET</a> source release at codeplex[*1]. Since this includes System.Json, I ported this to our class libs to replace existing stuff.<br>

<br>
There are only a few concerns.<br>
<br>
- There was some weird test that expects *invalid* JSON syntax.<br>
  I'm totally inclined to kill it, but since this says as if<br>
  .NET allowed such broken JSON (this actually fails with this<br>
  Microsoft implementation), I'd wait for inputs regarding this one<br>
  for a few days. Looks like it was from Miguel [*2]<br>
- I haven't verified builds other than net_2_0, net_4_0 and net_4_5.<br>
  I have excluded DLR stuff with NET_4_0 so this should build on<br>
  other profiles though.<br>
- The new dll is binary incompatible. I believe that in general<br>
  Microsoft implementation makes more sense though (lots of explicit<br>
  operators instead of implicit, Save() only on JsonValue, no<br>
  TimeSpan support).<br>
<br>
The new sources are in mcs/class/System.Json-new so the existing<br>
System.Jsoncan be easily replaced with it.<br>
<br>
[*1] <a href="http://aspnetwebstack.codeplex.com" target="_blank">http://aspnetwebstack.<u></u>codeplex.com</a><br>
[*2] <a href="https://github.com/mono/mono/blame/master/mcs/class/System.Json/Test/System.Json/JsonValueTest.cs" target="_blank">https://github.com/mono/mono/<u></u>blame/master/mcs/class/System.<u></u>Json/Test/System.Json/<u></u>JsonValueTest.cs</a><br>

<br>
Atsushi Eno<br>
<br>
______________________________<u></u>_________________<br>
Mono-devel-list mailing list<br>
<a href="mailto:Mono-devel-list@lists.ximian.com" target="_blank">Mono-devel-list@lists.ximian.<u></u>com</a><br>
<a href="http://lists.ximian.com/mailman/listinfo/mono-devel-list" target="_blank">http://lists.ximian.com/<u></u>mailman/listinfo/mono-devel-<u></u>list</a><br>
</blockquote></div><br></div>