windowscodecs/tests: Actually test some return values (PVS-Studio).

oldstable
Michael Stefaniuc 2015-03-04 09:54:53 +01:00 committed by Alexandre Julliard
parent a0e28403d4
commit c2eb206f99
2 changed files with 12 additions and 0 deletions

View File

@ -80,6 +80,7 @@ static void test_decoder_info(void)
int i;
hr = get_component_info(&CLSID_WICBmpDecoder, &info);
ok(hr == S_OK, "CreateComponentInfo failed, hr=%x\n", hr);
hr = IWICComponentInfo_QueryInterface(info, &IID_IWICBitmapDecoderInfo, (void**)&decoder_info);
ok(hr == S_OK, "QueryInterface failed, hr=%x\n", hr);

View File

@ -965,6 +965,7 @@ static void test_metadata_png(void)
if (SUCCEEDED(hr))
{
hr = IWICMetadataReader_GetMetadataFormat(reader, &containerformat);
ok(hr == S_OK, "GetMetadataFormat failed, hr=%#x\n", hr);
ok(IsEqualGUID(&containerformat, &GUID_MetadataFormatChunktIME) ||
broken(IsEqualGUID(&containerformat, &GUID_MetadataFormatUnknown)) /* Windows XP */,
"unexpected container format\n");
@ -1109,6 +1110,7 @@ static void test_metadata_gif(void)
if (SUCCEEDED(hr))
{
hr = IWICMetadataReader_GetMetadataFormat(reader, &format);
ok(hr == S_OK, "GetMetadataFormat failed, hr=%#x\n", hr);
ok(IsEqualGUID(&format, &GUID_MetadataFormatLSD), /* Logical Screen Descriptor */
"wrong metadata format %s\n", wine_dbgstr_guid(&format));
@ -1157,6 +1159,7 @@ static void test_metadata_gif(void)
if (SUCCEEDED(hr))
{
hr = IWICMetadataReader_GetMetadataFormat(reader, &format);
ok(hr == S_OK, "GetMetadataFormat failed, hr=%#x\n", hr);
ok(IsEqualGUID(&format, &GUID_MetadataFormatIMD), /* Image Descriptor */
"wrong metadata format %s\n", wine_dbgstr_guid(&format));
@ -1210,6 +1213,7 @@ static void test_metadata_gif(void)
if (SUCCEEDED(hr))
{
hr = IWICMetadataReader_GetMetadataFormat(reader, &format);
ok(hr == S_OK, "GetMetadataFormat failed, hr=%#x\n", hr);
ok(IsEqualGUID(&format, &GUID_MetadataFormatLSD), /* Logical Screen Descriptor */
"wrong metadata format %s\n", wine_dbgstr_guid(&format));
@ -1228,6 +1232,7 @@ static void test_metadata_gif(void)
if (SUCCEEDED(hr))
{
hr = IWICMetadataReader_GetMetadataFormat(reader, &format);
ok(hr == S_OK, "GetMetadataFormat failed, hr=%#x\n", hr);
ok(IsEqualGUID(&format, &GUID_MetadataFormatAPE), /* Application Extension */
"wrong metadata format %s\n", wine_dbgstr_guid(&format));
@ -1246,6 +1251,7 @@ static void test_metadata_gif(void)
if (SUCCEEDED(hr))
{
hr = IWICMetadataReader_GetMetadataFormat(reader, &format);
ok(hr == S_OK, "GetMetadataFormat failed, hr=%#x\n", hr);
ok(IsEqualGUID(&format, &GUID_MetadataFormatGifComment), /* Comment Extension */
"wrong metadata format %s\n", wine_dbgstr_guid(&format));
@ -1264,6 +1270,7 @@ static void test_metadata_gif(void)
if (SUCCEEDED(hr))
{
hr = IWICMetadataReader_GetMetadataFormat(reader, &format);
ok(hr == S_OK, "GetMetadataFormat failed, hr=%#x\n", hr);
ok(IsEqualGUID(&format, &GUID_MetadataFormatUnknown),
"wrong metadata format %s\n", wine_dbgstr_guid(&format));
@ -1312,6 +1319,7 @@ static void test_metadata_gif(void)
if (SUCCEEDED(hr))
{
hr = IWICMetadataReader_GetMetadataFormat(reader, &format);
ok(hr == S_OK, "GetMetadataFormat failed, hr=%#x\n", hr);
ok(IsEqualGUID(&format, &GUID_MetadataFormatIMD), /* Image Descriptor */
"wrong metadata format %s\n", wine_dbgstr_guid(&format));
@ -1330,6 +1338,7 @@ static void test_metadata_gif(void)
if (SUCCEEDED(hr))
{
hr = IWICMetadataReader_GetMetadataFormat(reader, &format);
ok(hr == S_OK, "GetMetadataFormat failed, hr=%#x\n", hr);
ok(IsEqualGUID(&format, &GUID_MetadataFormatGifComment), /* Comment Extension */
"wrong metadata format %s\n", wine_dbgstr_guid(&format));
@ -1349,6 +1358,7 @@ static void test_metadata_gif(void)
if (SUCCEEDED(hr))
{
hr = IWICMetadataReader_GetMetadataFormat(reader, &format);
ok(hr == S_OK, "GetMetadataFormat failed, hr=%#x\n", hr);
ok(IsEqualGUID(&format, &GUID_MetadataFormatUnknown),
"wrong metadata format %s\n", wine_dbgstr_guid(&format));
@ -1367,6 +1377,7 @@ static void test_metadata_gif(void)
if (SUCCEEDED(hr))
{
hr = IWICMetadataReader_GetMetadataFormat(reader, &format);
ok(hr == S_OK, "GetMetadataFormat failed, hr=%#x\n", hr);
ok(IsEqualGUID(&format, &GUID_MetadataFormatGCE), /* Graphic Control Extension */
"wrong metadata format %s\n", wine_dbgstr_guid(&format));