Re: [PATCH 3/5] atl: Added stub AtlIPersistPropertyBag_Save.

[Jacek Caban]

On 01/23/15 10:18, Qian Hong wrote:
> +HRESULT WINAPI AtlIPersistPropertyBag_Save(LPPROPERTYBAG pPropBag, BOOL fClearDirty, BOOL fSaveAllProperties,
> -
>                                       ATL_PROPMAP_ENTRY *pMap, void *pThis, IUnknown *pUnk)
>
> +{
> -
> - FIXME("(%p, %x, %x, %p, %p, %p)\n", pPropBag, fClearDirty, fSaveAllProperties, pMap, pThis, pUnk);
> - return S_OK;
>
> +}

It's a minor detail, but after two patches in series removing
unnecessary empty lines, you add a new one here ;)

Cheers,
Jacek

[PATCH 3/5] atl: Added stub AtlIPersistPropertyBag_Save.

[Qian Hong]

On Fri, Jan 23, 2015 at 6:03 PM, Jacek Caban  wrote:
> after two patches in series removing
> unnecessary empty lines, you add a new one here ;)

Intentionally, this is incremental development mode <grin>