Hi,
This looks ok, I'm just concerned that the new code will be run even
in non-com situations,
and it might cause problems, like it uses CEE_LDIND_I4 to load a bool value.
Zoltan
2009/2/12 Bill Holmes <billholme...@gmail.com>:
> Hi,
>
> The attached patch fixes some problems we are seeing with marshaling
> bools and IDspatch types in Native code.
>
> I can split the patch separating the bool changes form the dispatch
> changes if needed. I would like to apply this to the 2.4 branch as
> well.
>
> -bill
>
> 2009-02-12 Bill Holmes <billholme...@gmail.com>
>
> * object-internals.h : Fixing a typo in the
> MonoReflectionComVisibleAttribute struct.
>
> * marshal.c (cominterop_com_visible): Check the implemented
> interfaces for ComImport.
>
> * marshal.c (cominterop_get_native_wrapper_adjusted): For COM calls
> assume that bools should be treated as VARIANTBOOLs.
>
> * marshal.c (emit_marshal_boolean): Adding cases for
> MARSHAL_ACTION_MANAGED_CONV_IN and MARSHAL_ACTION_MANAGED_CONV_OUT.
>
> * marshal.c (mono_marshal_emit_managed_wrapper): Adding calls to
> emit_marshal MARSHAL_ACTION_MANAGED_CONV_IN and OUT for bools.
>
> * marshal.c (cominterop_get_ccw): For COM calls assume that bools
> should be treated as VARIANTBOOLs.
>
> Code is contributed under MIT/X11 license.
>
> _______________________________________________
> Mono-devel-list mailing list
> Mono-devel-list@lists.ximian.com
> http://lists.ximian.com/mailman/listinfo/mono-devel-list
>
>
_______________________________________________
Mono-devel-list mailing list
Mono-devel-list@lists.ximian.com
http://lists.ximian.com/mailman/listinfo/mono-devel-list
