<div dir="ltr">Applied!</div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Oct 30, 2014 at 2:01 PM, Alexander Köplinger <span dir="ltr"><<a href="mailto:alex.koeplinger@outlook.com" target="_blank">alex.koeplinger@outlook.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


<div><div dir="ltr"><font color="#000000" face="Calibri,sans-serif">:-( The PR that fixes the test failures was submitted as <a href="https://github.com/mono/mono/pull/1380" target="_blank">https://github.com/mono/mono/pull/1380</a></font><br><font color="#000000" face="Calibri,sans-serif">Can you revert the revert and apply the patch instead?</font><br> <br>-- Alex<br><font color="#000000" face="Calibri,sans-serif"><br></font><br> <br><div><hr>From: <a href="mailto:miguel@xamarin.com" target="_blank">miguel@xamarin.com</a><br>Date: Thu, 30 Oct 2014 13:54:48 -0400<span class=""><br>Subject: Re: [Mono-dev] PR #1337 Review<br></span>To: <a href="mailto:alex.koeplinger@outlook.com" target="_blank">alex.koeplinger@outlook.com</a><br>CC: <a href="mailto:ryan.melena@noesislabs.com" target="_blank">ryan.melena@noesislabs.com</a>; <a href="mailto:mono-devel-list@lists.ximian.com" target="_blank">mono-devel-list@lists.ximian.com</a><div><div class="h5"><br><br><div dir="ltr">I have reverted the patch for now.<div><br></div><div>Miguel</div></div><div><br><div>On Thu, Oct 30, 2014 at 12:49 PM, Alexander Köplinger <span dir="ltr"><<a href="mailto:alex.koeplinger@outlook.com" target="_blank">alex.koeplinger@outlook.com</a>></span> wrote:<br><blockquote style="padding-left:1ex;border-left-color:rgb(204,204,204);border-left-width:1px;border-left-style:solid">


<div><div dir="ltr"><font color="#000000" face="Calibri,sans-serif">Thanks for looking into it!<br></font><br> <br><div><hr>Date: Thu, 30 Oct 2014 11:47:01 -0500<span><br>Subject: Re: [Mono-dev] PR #1337 Review<br>From: <a href="mailto:ryan.melena@noesislabs.com" target="_blank">ryan.melena@noesislabs.com</a><br>To: <a href="mailto:alex.koeplinger@outlook.com" target="_blank">alex.koeplinger@outlook.com</a><br><br></span><div dir="ltr">Turns out the IEnumerable<object> vs ArrayList issue was likely caused by an error in our initial Deserialize(string, Type) method implementation that we misdiagnosed as an issue with using ArrayList.  Working on a pull request to revert the ArrayList / List<object> changes to JavaScriptSerializer.cs file.<div><br></div><div>Also found the source of the new unit test failure (an issue with our newest Deserialize(string, Type) method implementation) and will include a fix for that in the next pull request as well.</div></div></div>                                           </div></div>
<br>_______________________________________________<br>
Mono-devel-list mailing list<br>
<a href="mailto:Mono-devel-list@lists.ximian.com" target="_blank">Mono-devel-list@lists.ximian.com</a><br>
<a href="http://lists.ximian.com/mailman/listinfo/mono-devel-list" target="_blank">http://lists.ximian.com/mailman/listinfo/mono-devel-list</a><br>
<br></blockquote></div><br></div></div></div></div>                                       </div></div>
</blockquote></div><br></div>