Re: [PATCH v8 0/4] MR7928: coremessaging: Partially implement CreateDispatcherQueueController().

[Nikolay Sivov (＠nsivov)]

Nikolay Sivov (@nsivov) commented about dlls/coremessaging/tests/coremessaging.c:
> - if (hr == REGDB_E_CLASSNOTREG)
> - {
> -
>    win_skip( "%s runtimeclass not registered, skipping tests.\n", wine_dbgstr_w( dispatcher_queue_controller_statics_name ) );
> -
>    return;
> - }
> -
> - check_interface( factory, &IID_IUnknown );
> - check_interface( factory, &IID_IInspectable );
> - check_interface( factory, &IID_IAgileObject );
> -
> - hr = IActivationFactory_QueryInterface( factory, &IID_IDispatcherQueueControllerStatics, (void **)&dispatcher_queue_controller_statics );
> - ok( hr == S_OK, "got hr %#lx.\n", hr );
> -
> - hr = IDispatcherQueueControllerStatics_CreateOnDedicatedThread( dispatcher_queue_controller_statics, NULL );
> - todo_wine
> - ok( hr == E_POINTER || hr == 0x80000005 /* E_POINTER #if !defined(_WIN32) || defined(_MAC) */, "got hr %#lx.\n", hr );

Just use FAILED(hr), or don't test for NULL argument case at all.