Re: [PATCH v2 0/1] MR8456: urlmon: Add PersistentZoneIdentifier COM object stubs

[Jacek Caban (＠jacek)]

Jacek Caban (@jacek) commented about dlls/urlmon/zone_id.c:
> +{
> - return CONTAINING_RECORD(iface, PersistentZoneIdentifier, IZoneIdentifier_iface);
>
> +}
> -
>
> +static HRESULT WINAPI PZIUnk_QueryInterface(IUnknown *iface, REFIID riid, void **ppv)
> +{
> - PersistentZoneIdentifier *This = impl_from_IUnknown(iface);
> -
> - *ppv = NULL;
> -
> - if (IsEqualGUID(&IID_IUnknown, riid))
> - {
> -
>    TRACE("(%p)->(IID_IUnknown %p)\n", This, ppv);
> -
>    *ppv = &This->IUnknown_inner;
> - } else if (IsEqualGUID(&IID_IPersist, riid))
> - {

Please use a consistent style: either place both the opening and closing brackets on their own lines, or keep both inline, but don’t mix the two.