[Mono-dev] PR #1337 Review

Alexander Köplinger alex.koeplinger at outlook.com
Thu Oct 30 13:23:09 UTC 2014


Hey, this breaks a few tests:
 
MonoTests.System.Web.Script.Serialization.JavaScriptSerializerTest.DeserializeDictionaryOfArrayList MonoTests.System.Web.Script.Serialization.JavaScriptSerializerTest.DeserializeObject MonoTests.System.Web.Script.Serialization.JavaScriptSerializerTest.InfinityAndNaN MonoTests.System.Web.Script.Serialization.JavaScriptSerializerTest.TestDeserialize MonoTests.System.Web.Script.Serialization.JavaScriptSerializerTest.TestDeserializeConverter1 MonoTests.System.Web.Script.Serialization.JavaScriptSerializerTest.TestDeserializeNonGenericOverload MonoTests.System.Web.Script.Serialization.JavaScriptSerializerTest.TestDeserializeTypeResolver MonoTests.System.Web.Script.Serialization.JavaScriptSerializerTest.TestSerialize1 
 
The TestDeserializeNonGenericOverload is a new one added by the PR (it passes on MS.NET though, so we should make it work on Mono).
 
I'm not sure the switch from ArrayList to List<object> is correct, if I change that back all tests pass again except the new TestDeserializeNonGenericOverload.
 
-- Alex

 
From: miguel at xamarin.com
Date: Wed, 29 Oct 2014 15:02:39 -0400
To: Ryan.Melena+Mono-Devel-List at noesislabs.com
CC: mono-devel-list at lists.ximian.com
Subject: Re: [Mono-dev] PR #1337 Review

Thanks for pointing this out.
Imlemented.
Miguel
On Wed, Oct 29, 2014 at 2:52 PM, RyanMelenaNoesis <Ryan.Melena+Mono-Devel-List at noesislabs.com> wrote:
It was requested that I notify the mailing list about this pull request ( https://github.com/mono/mono/pull/1337 ) in order to facilitate conversation.  The changes were implemented to support JWT authentication in our web application and required a couple changes to existing Mono code.  The changes to existing code were necessary to make Mono compatible with MS calling code and are explained in line comments.  Please let me know if anyone has particular concerns with this pull request.

_______________________________________________

Mono-devel-list mailing list

Mono-devel-list at lists.ximian.com

http://lists.ximian.com/mailman/listinfo/mono-devel-list





_______________________________________________
Mono-devel-list mailing list
Mono-devel-list at lists.ximian.com
http://lists.ximian.com/mailman/listinfo/mono-devel-list 		 	   		  
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.ximian.com/pipermail/mono-devel-list/attachments/20141030/89c1658f/attachment.html>


More information about the Mono-devel-list mailing list