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.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/8459#note_109714