Re: [PATCH v4 0/5] MR3519: windows.devices.enumeration: Implement IDeviceInformationStatics::CreateWatcherAqsFilter().

[Rémi Bernon (＠rbernon)]

On Wed Aug  9 04:43:08 2023 +0000, Mohamad Al-Jaf wrote:
> I thought I did add `WindowsDeleteString`. I wasn't sure if adding
> `WindowsDuplicateString` to `device_statics2_CreateWatcher` should be in
> the same commit or not. I added it in a new commit.
> Thanks for the review.

If you want to put it in a separate commit, it should probably come first, with the field addition and the `WindowsDeleteString` call. Otherwise when `device_statics2_CreateWatcher` was used, `Release` will call `WindowsDeleteString` on destruction with an invalid filter (though perhaps zero-initialized HSTRING is valid, but imho it'd be better that way).