Michael Stefaniuc : windowscodecs/tests: Actually test some return values (PVS-Studio).
Alexandre Julliard
julliard at wine.codeweavers.com
Wed Mar 4 09:22:17 CST 2015
Module: wine
Branch: master
Commit: c2eb206f99443ecfb29f347e5c753d97944b5b14
URL: http://source.winehq.org/git/wine.git/?a=commit;h=c2eb206f99443ecfb29f347e5c753d97944b5b14
Author: Michael Stefaniuc <mstefani at redhat.de>
Date: Wed Mar 4 09:54:53 2015 +0100
windowscodecs/tests: Actually test some return values (PVS-Studio).
---
dlls/windowscodecs/tests/info.c | 1 +
dlls/windowscodecs/tests/metadata.c | 11 +++++++++++
2 files changed, 12 insertions(+)
diff --git a/dlls/windowscodecs/tests/info.c b/dlls/windowscodecs/tests/info.c
index 51181ca..8a22970 100644
--- a/dlls/windowscodecs/tests/info.c
+++ b/dlls/windowscodecs/tests/info.c
@@ -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);
diff --git a/dlls/windowscodecs/tests/metadata.c b/dlls/windowscodecs/tests/metadata.c
index fa9d701..20fe54c 100644
--- a/dlls/windowscodecs/tests/metadata.c
+++ b/dlls/windowscodecs/tests/metadata.c
@@ -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));
More information about the wine-cvs
mailing list