[Mono-dev] BSTR Patch

[Zoltan Varga]

                              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
>
>
>
>