The icall removal patch is ok.<br><br>The second one is tricky. Do we really want to completely disable COM support when running under the sandbox?<br>It does make sense for moonlight, but not for other users of coreclr.<br>
<br>I believe we should only fail COM for non-platform assemblies which has the same result for moonlight but won&#39;t<br>bite future users of the sandbox code.<br><br><br><br><div class="gmail_quote">On Thu, Oct 29, 2009 at 4:43 PM, Sebastien Pouliot <span dir="ltr">&lt;<a href="mailto:sebastien@ximian.com">sebastien@ximian.com</a>&gt;</span> wrote:<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">Hello,<br>
<br>
Two small/easy patches for review.<br>
<br>
The first one avoid calling mono_com_init when coreclr is enabled*.<br>
This avoid a crash if some assembly use [ComImport] on a type and throw<br>
a TypeLoadException - which is what happens in Silverlight.<br>
<br>
        * For some reason (I guess it use COM for it&#39;s platform code,<br>
        while Moonlight does not) Silverlight expose [ComImport] but<br>
        otherwise does not support COM (as least for application code).<br>
<br>
Second patch removes some internal calls (all strings except one) that<br>
are not used (anymore) in the class libraries.<br>
<font color="#888888"><br>
Sebastien<br>
</font><br>
p.s. both patches were created from 2-6 branch but I&#39;ll commit them<br>
against HEAD too.<br>
<br>_______________________________________________<br>
Mono-devel-list mailing list<br>
<a href="mailto:Mono-devel-list@lists.ximian.com">Mono-devel-list@lists.ximian.com</a><br>
<a href="http://lists.ximian.com/mailman/listinfo/mono-devel-list" target="_blank">http://lists.ximian.com/mailman/listinfo/mono-devel-list</a><br>
<br></blockquote></div><br>