[Mono-dev] BSTR Patch

Zoltan Varga vargaz at gmail.com
Tue May 2 14:54:45 EDT 2006


                              Hi,

  The patch looks good and IMHO it can be committed as it is. As for the tests,
they should go to either one of the marshalX.cs files in tests/, or to a new
file, it doesn't really matter. Care should be taken to only run them on MS.NET,
using some kind of dynamic check.

                                  Zoltan

On 5/2/06, Jonathan S. Chambers <Jonathan.Chambers at ansys.com> wrote:
> Here's the first of some COM Interop patches; this one adds BSTR support (on Windows only). I'll send some test patches too, if someone could tell me whether they should be part of some current test or a new one.
>
> On platforms other than Windows the BSTR related methods currently g_error (this was the old code). Should I throw a NotSupported/NotImplemented exception instead?
>
> Thanks,
> Jonathan
>
>
>
> _______________________________________________
> Mono-devel-list mailing list
> Mono-devel-list at lists.ximian.com
> http://lists.ximian.com/mailman/listinfo/mono-devel-list
>
>
>
>



More information about the Mono-devel-list mailing list