<html>
<head>
<style><!--
.hmmessage P
{
margin:0px;
padding:0px
}
body.hmmessage
{
font-size: 12pt;
font-family:Calibri
}
--></style></head>
<body class='hmmessage'><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">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 id="FontBreak"></font><br> <BR><div><hr id="stopSpelling">From: miguel@xamarin.com<br>Date: Thu, 30 Oct 2014 13:54:48 -0400<br>Subject: Re: [Mono-dev] PR #1337 Review<br>To: alex.koeplinger@outlook.com<br>CC: ryan.melena@noesislabs.com; mono-devel-list@lists.ximian.com<br><br><div dir="ltr">I have reverted the patch for now.<div><br></div><div>Miguel</div></div><div class="ecxgmail_extra"><br><div class="ecxgmail_quote">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 class="ecxgmail_quote" 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">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></body>
</html>