[PATCH] strmbase: Display dll in which we are searching a class to instanciate.
Christian Costa
titan.costa at gmail.com
Wed Mar 28 07:33:34 CDT 2012
2012/3/28 Alexandre Julliard <julliard at winehq.org>
> Christian Costa <titan.costa at gmail.com> writes:
>
> > 2012/3/28 Alexandre Julliard <julliard at winehq.org>
> >
> >> Christian Costa <titan.costa at gmail.com> writes:
> >>
> >> > @@ -360,14 +361,17 @@ HRESULT WINAPI
> STRMBASE_DllGetClassObject(REFCLSID
> >> rclsid, REFIID riid, LPVOID *
> >> > break;
> >> > }
> >> >
> >> > + if (!GetModuleFileNameA(g_hInst, dllname, sizeof(dllname)))
> >> > + strcpy(dllname, "???");
> >>
> >> You don't want to do that in the success case.
> >>
> >>
> > GetModuleFileNameA returns the length of the string copied to the buffer.
> > The length is 0 in case of error.
>
> What I mean is that you only need the module name in case the class is
> not found (and really, only in the null constructor case).
>
>
It's convenient to see from a log where the class is supposed to be
implemented but since this can be retreive in the registry, it can be
remove.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.winehq.org/pipermail/wine-devel/attachments/20120328/d6bccbb2/attachment.html>
More information about the wine-devel
mailing list