[PATCH 5/7] npmshtml: Added Mozilla plugin for mshtml ActiveX controls.

Alexandre Julliard julliard at winehq.org
Thu Dec 2 07:23:47 CST 2010


Jacek Caban <jacek at codeweavers.com> writes:

> Um, that was stupid from me. I've previously made NP_* function
> failing, but it didn't work well, so I've changed it to DllMain. I
> don't see a clean way to fix registration. We could set these
> registries from mshtml.dll, but then we lose the ability to easily
> unregister npmshtml.dll.
>
> I've sent a patch that checks if it's running from rundll32 or
> regsvr32 and allows the DLL to load them. I hope it's not too hackish.

I don't think it's a good idea to try to magically fail on DLL load.  It
would prevent registering from other tools, loading to check version
resources, etc. It's preferable to always allow it to load.

Of course an even better way would be to change Gecko to make it
possible to not have a separate dll at all...

-- 
Alexandre Julliard
julliard at winehq.org



More information about the wine-devel mailing list