[Mono-dev] [PATCH] Managed Marshal.Copy implemantations

[Kornél Pál]

On Dec 15, 2007 2:09 AM, Felipe Lessa <felipe.lessa at gmail.com> wrote:
> On Dec 14, 2007 10:59 PM, Kornél Pál <kornelpal at gmail.com> wrote:
>> I see no reason to implement Marshal.Copy in unmanaged code. I'm wrong
>> please let me know.
>
> Well, it seems like there's a lot more duplicated code in your patch
> than before.

Only argument checks are redundant, but I think argument checks are 
redundant in the whole class library.

The code that invokes memcpy is different for each method because of 
different types used.

Can I commit this patch or should Marshal.Copy remain unmanaged or do you 
have any other objections?

Kornél 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: Marshal.diff
Type: application/octet-stream
Size: 18335 bytes
Desc: not available
Url : http://lists.ximian.com/pipermail/mono-devel-list/attachments/20071217/4aa73b41/attachment.obj