[PATCH 0/1] MR4963: windowscodecs: Pass result as void pointer to ComponentInfo_GetUINTValue.

[Jacek Caban (＠jacek)]

It's often just casted by callers anyway. Fixes clang warning:
```
dlls/windowscodecs/info.c:1426:81: warning: passing 'WICPixelFormatNumericRepresentation *' (aka 'enum WICPixelFormatNumericRepresentation *') to parameter of type 'UINT *' (aka 'unsigned int *') converts between pointers to integer types with different sign [-Wpointer-sign]
    return ComponentInfo_GetUINTValue(This->classkey, L"NumericRepresentation", pNumericRepresentation);
                                                                                ^~~~~~~~~~~~~~~~~~~~~~
../dlls/windowscodecs/info.c:104:11: note: passing argument to parameter 'result' here
    UINT *result)
          ^
```

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/4963

[PATCH 1/1] windowscodecs: Pass result as void pointer to ComponentInfo_GetUINTValue.

[Jacek Caban]

From: Jacek Caban <jacek(a)codeweavers.com>

---
 dlls/windowscodecs/info.c | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/dlls/windowscodecs/info.c b/dlls/windowscodecs/info.c
index 689d2e26314..4edfaa69d77 100644
--- a/dlls/windowscodecs/info.c
+++ b/dlls/windowscodecs/info.c
@@ -101,7 +101,7 @@ static HRESULT ComponentInfo_GetGUIDValue(HKEY classkey, LPCWSTR value,
 }
 
 static HRESULT ComponentInfo_GetUINTValue(HKEY classkey, LPCWSTR value,
-    UINT *result)
+    void *result)
 {
     LONG ret;
     DWORD cbdata = sizeof(DWORD);
@@ -110,11 +110,11 @@ static HRESULT ComponentInfo_GetUINTValue(HKEY classkey, LPCWSTR value,
         return E_INVALIDARG;
 
     ret = RegGetValueW(classkey, NULL, value, RRF_RT_DWORD, NULL,
-        (DWORD *)result, &cbdata);
+        result, &cbdata);
 
     if (ret == ERROR_FILE_NOT_FOUND)
     {
-        *result = 0;
+        *(UINT *)result = 0;
         return S_OK;
     }
 
@@ -1413,7 +1413,7 @@ static HRESULT WINAPI PixelFormatInfo_SupportsTransparency(IWICPixelFormatInfo2
 
     TRACE("(%p,%p)\n", iface, pfSupportsTransparency);
 
-    return ComponentInfo_GetUINTValue(This->classkey, L"SupportsTransparency", (UINT *)pfSupportsTransparency);
+    return ComponentInfo_GetUINTValue(This->classkey, L"SupportsTransparency", pfSupportsTransparency);
 }
 
 static HRESULT WINAPI PixelFormatInfo_GetNumericRepresentation(IWICPixelFormatInfo2 *iface,
@@ -1682,7 +1682,7 @@ static HRESULT WINAPI MetadataReaderInfo_DoesRequireFullStream(IWICMetadataReade
 {
     MetadataReaderInfo *This = impl_from_IWICMetadataReaderInfo(iface);
     TRACE("(%p,%p)\n", iface, param);
-    return ComponentInfo_GetUINTValue(This->classkey, L"RequiresFullStream", (UINT *)param);
+    return ComponentInfo_GetUINTValue(This->classkey, L"RequiresFullStream", param);
 }
 
 static HRESULT WINAPI MetadataReaderInfo_DoesSupportPadding(IWICMetadataReaderInfo *iface,
@@ -1690,7 +1690,7 @@ static HRESULT WINAPI MetadataReaderInfo_DoesSupportPadding(IWICMetadataReaderIn
 {
     MetadataReaderInfo *This = impl_from_IWICMetadataReaderInfo(iface);
     TRACE("(%p,%p)\n", iface, param);
-    return ComponentInfo_GetUINTValue(This->classkey, L"SupportsPadding", (UINT *)param);
+    return ComponentInfo_GetUINTValue(This->classkey, L"SupportsPadding", param);
 }
 
 static HRESULT WINAPI MetadataReaderInfo_DoesRequireFixedSize(IWICMetadataReaderInfo *iface,
-- 
GitLab

https://gitlab.winehq.org/wine/wine/-/merge_requests/4963

[PATCH 0/1] MR4963: windowscodecs: Pass result as void pointer to ComponentInfo_GetUINTValue. - approved

[Esme Povirk (＠madewokherd)]

This merge request was approved by Esme Povirk.
-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/4963