Detlef Riekenberg <wine....@web.de> 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. -- Dmitry.
