----- Original Message ----- 
From: "Jacek Caban" <jack(a)itma.pwr.wroc.pl>
> @@ -78,29 +77,35 @@ static HRESULT WINAPI PersistMoniker_IsD
>  static HRESULT WINAPI PersistMoniker_Load(IPersistMoniker *iface, BOOL
fFullyAvailable,
>          IMoniker *pimkName, LPBC pibc, DWORD grfMode)
>  {
> -    FIXME("(%p)->(%x %p %p %08lx)\n", iface, fFullyAvailable, pimkName,
pibc, grfMode);
> +    PERSISTMON_THIS
> +    FIXME("(%p)->(%x %p %p %08lx)\n", This, fFullyAvailable, pimkName,
pibc, grfMode);
>      return S_OK;
>  }
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.
-- 
Dimi Paun <dimi(a)lattica.com>
Lattica, Inc.