Re: [PATCH v6 0/5] MR1588: Add windows.system.profile.systemmanufacturers DLL.

[Mohamad Al-Jaf (＠maljaf)]

On Tue Dec  6 06:29:13 2022 +0000, Rémi Bernon wrote:
> It's a bit annoying that the test crashes on Win8, the `if (0)` makes it
> pretty useless to have. However it looks like it never succeeds anyway,
> and the wbem query returns `WBEM_E_ACCESS_DENIED` on every VM. I suspect
> it was badly handled in Win8, hence the crash, and that got fixed later,
> returning E_UNEXPECTED.

Yeah, it's a bit surprising how WinRT handles errors or rather doesn't in some cases. I guess it was rushed with Windows 8.

Thanks for the review!