Re: [PATCH v7 0/2] MR8459: urlmon: Add PersistentZoneIdentifier semi-stub

[Jacek Caban (＠jacek)]

Jacek Caban (@jacek) commented about dlls/urlmon/tests/zone_id.c:
> -
>
> +typedef struct load_file_test {
> - LPCWSTR name;
> - DWORD mode;
> - HRESULT hres;
>
> +} load_file_test;
> -
>
> +typedef struct _save_file_test {
> - LPCWSTR name;
> - BOOL remember;
> - HRESULT hres;
>
> +} save_file_test;
> -
>
> +typedef struct _persist_file_test {
> - zone_id_op_test set;
> - HRESULT is_dirty_before_save;

All those `is_dirty_*` do not seem very useful if they are always `S_OK` anyway. You may just as well just use `S_OK` in the test code.