Re: [PATCH 0/10] MR1537: Draft: windows.media: Partially implement caption properties.

[Rémi Bernon (＠rbernon)]

Rémi Bernon (@rbernon) commented about dlls/windows.media/tests/captions.c:
>  hr = IActivationFactory_QueryInterface( factory, &IID_IClosedCaptionPropertiesStatics, (void **)&caption_statics );
>  ok( hr == S_OK, "got hr %#lx.\n", hr );
>
> - if (0) /* Crash on Windows */
> - {
> -
>    hr = IClosedCaptionPropertiesStatics_get_FontColor( caption_statics, NULL );
> -
>    todo_wine ok( hr == S_OK, "get_FontColor returned %#lx\n", hr );
> - }

If you've tested that the pointers aren't validated on Windows I think you can just omit the tests.

Sometimes it is an important information and we have such tests, but most of the time it isn't.

Same for all the other functions below.