Bug 323072 (MONO80383) - [PATCH] support for generic types & methods in remoting interfaces
Summary: [PATCH] support for generic types & methods in remoting interfaces
Status: RESOLVED FIXED
Alias: MONO80383
Product: Mono: Runtime
Classification: Mono
Component: remoting (show other bugs)
Version: 1.2
Hardware: Other All
: P3 - Medium : Major
Target Milestone: ---
Assignee: Lluis Sanchez
QA Contact: Mono Bugs
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-12-28 07:02 UTC by Robert Jordan
Modified: 2007-09-15 21:24 UTC (History)
0 users

See Also:
Found By: ---
Services Priority:
Business Priority:
Blocker: ---
Marketing QA Status: ---
IT Deployment: ---


Attachments
corlib_remoting_20.diff (14.01 KB, patch)
2006-12-28 07:03 UTC, Thomas Wiest
Details | Diff
System.Runtime.Remoting.Test.GenericTest.cs - new file (1.84 KB, text/plain)
2006-12-28 07:22 UTC, Thomas Wiest
Details
System.Runtime.Remoting_test.dll.sources.diff (330 bytes, patch)
2006-12-28 07:23 UTC, Thomas Wiest
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Thomas Wiest 2007-09-15 20:21:13 UTC


---- Reported by robertj@gmx.net 2006-12-28 00:02:55 MST ----

The patches consists of:

corlib/System.Runtime.Serialization.Formatters.Binary:

- documentation update of the binary serialization format

- BinaryCommon.Methodflags update

- MessageFormater update

- ObjectWriter.cs, ObjectReader.cs:

  MS.NET serializes generic parameters as System.RuntimeType objects.
  Because Mono doesn't know this type, the binary formatter must
  map between System.RuntimeType and System.MonoType.

  This (maybe questionable) change is only required for interoperability.
  I tried to make the patch less intrusive by checking the serialization
  context.


corlib/System.Runtime.Remoting.Messaging:

- MethodCall: fix a buglet that prevented CastTo() to handle closed
  generic types correctly.

- MethodCall: add support for generic method arguments

- ReturnMessage: fix for https://bugzilla.novell.com/show_bug.cgi?id=MONO80371.


System.Runtime.Remoting/Tests:

- GenericTest: unit tests. I'll try later to integrate this test into
  the existing tests, if possible.



---- Additional Comments From robertj@gmx.net 2006-12-28 00:03:41 MST ----

Created an attachment (id=171182)
corlib_remoting_20.diff




---- Additional Comments From robertj@gmx.net 2006-12-28 00:13:08 MST ----

Note that the SOAP formatter remains unchanged because MS.NET's own
formatter doesn't support generic types.

Mono's supports open generic types, but I'll add a check&trow
for this to match MS.




---- Additional Comments From robertj@gmx.net 2006-12-28 00:22:06 MST ----

Created an attachment (id=171183)
System.Runtime.Remoting.Test.GenericTest.cs  - new file




---- Additional Comments From robertj@gmx.net 2006-12-28 00:23:29 MST ----

Created an attachment (id=171184)
System.Runtime.Remoting_test.dll.sources.diff




---- Additional Comments From lluis@ximian.com 2007-01-08 09:00:11 MST ----

The patch looks ok to me. Feel free to commit. Thanks!



---- Additional Comments From robertj@gmx.net 2007-01-08 09:09:42 MST ----

May I normalize the line endings before committing the changes?




---- Additional Comments From miguel@ximian.com 2007-01-08 14:34:16 MST ----

Robert, that sounds like a good idea.

Go ahead.



---- Additional Comments From robertj@gmx.net 2007-01-08 20:45:32 MST ----

Fixed in SVN r70683-70684.




---- Additional Comments From robertj@gmx.net 2007-01-08 20:46:47 MST ----

Forget to close.


This bug blocked bug(s) 78882.
Imported an attachment (id=171182)
Imported an attachment (id=171183)
Imported an attachment (id=171184)