MSHTML: THIS macros clean up
Jacek Caban
jack at itma.pwr.wroc.pl
Tue Jun 28 17:49:50 CDT 2005
Dimi Paun wrote:
>
>This code looks weird. It should still look like C code,
>even if we use a macro. I would suggest something like this:
>
>static HRESULT WINAPI PersistMoniker_Load(IPersistMoniker *iface, BOOL
>fFullyAvailable,
> IMoniker *pimkName, LPBC pibc, DWORD grfMode)
>{
> HTMLDocument *This = PERSISTMON_THIS(iface);
>
> FIXME("(%p)->(%x %p %p %08lx)\n", This, fFullyAvailable, pimkName, pibc,
>grfMode);
> return S_OK;
>}
>
>It's a bit more verbose, but a lot more readable.
>
>
>
Ok, it really looks better. Problem is that macros like
PERSISTMON_THIS are in whole mshtml so I'll have to
clean up much more. I'll do so later.
Alexandre: please don't apply this patch.
Thanks,
Jacek
More information about the wine-patches
mailing list