Re: [PATCH] windowscodecs/tests: Fix a test failure [newwtb and other]

[Dmitry Timoshkov]

Detlef Riekenberg  wrote:
> - /* global palette */
> - /* global palette. This succeed since win7sp1 with platform update / win8 */

This comment is not helpful.
>  hr = IWICBitmapDecoder_CopyPalette(decoder, palette);
> - ok(hr == WINCODEC_ERR_FRAMEMISSING,
> -
>   "expected WINCODEC_ERR_FRAMEMISSING, got %#x\n", hr);
> - ok((hr == S_OK) || (hr == WINCODEC_ERR_FRAMEMISSING),
> -
>   "expected S_OK or WINCODEC_ERR_FRAMEMISSING, got %#x\n", hr);

One of test results should be marked as broken, and in the S_OK case
the resulting palette should be tested and released. I'd guess that newer
version should be taken as a model to follow, and Wine implementation
should be fixed appropriately.