system/wine
system/wine
1
0
Fork 0
mirror of git://source.winehq.org/git/wine.git synced 2024-09-15 07:40:21 +00:00
Code Issues Packages Projects Releases Wiki Activity

gdiplus: Use nameless unions.

Browse source 
Signed-off-by: Jacek Caban <jacek@codeweavers.com>
Signed-off-by: Esme Povirk <esme@codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard@winehq.org>
This commit is contained in:
Jacek Caban 2021-03-27 16:02:12 +01:00 committed by Alexandre Julliard
parent 23ebe1ed7f
commit 883298f627
2 changed files with 29 additions and 30 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
dlls/gdiplus/Makefile.in
View file

@ -2,7 +2,6 @@ MODULE = gdiplus.dll
IMPORTLIB = gdiplus IMPORTLIB = gdiplus
IMPORTS = uuid shlwapi ole32 user32 gdi32 IMPORTS = uuid shlwapi ole32 user32 gdi32
DELAYIMPORTS = windowscodecs DELAYIMPORTS = windowscodecs
EXTRADEFS = -DWINE_NO_NAMELESS_EXTENSION
EXTRADLLFLAGS = -mno-cygwin EXTRADLLFLAGS = -mno-cygwin

58
dlls/gdiplus/image.c
View file

@ -2467,7 +2467,7 @@ GpStatus WINGDIPAPI GdipGetPropertyIdList(GpImage *image, UINT num, PROPID *list
list[i] = 0; list[i] = 0;
continue; continue;
} }
list[i] = id.u.uiVal; list[i] = id.uiVal;
} }
IWICEnumMetadataItem_Release(enumerator); IWICEnumMetadataItem_Release(enumerator);
@ -2484,25 +2484,25 @@ static UINT propvariant_size(PROPVARIANT *value)
case VT_I1: case VT_I1:
case VT_UI1: case VT_UI1:
if (!(value->vt & VT_VECTOR)) return 1; if (!(value->vt & VT_VECTOR)) return 1;
return value->u.caub.cElems; return value->caub.cElems;
case VT_I2: case VT_I2:
case VT_UI2: case VT_UI2:
if (!(value->vt & VT_VECTOR)) return 2; if (!(value->vt & VT_VECTOR)) return 2;
return value->u.caui.cElems * 2; return value->caui.cElems * 2;
case VT_I4: case VT_I4:
case VT_UI4: case VT_UI4:
case VT_R4: case VT_R4:
if (!(value->vt & VT_VECTOR)) return 4; if (!(value->vt & VT_VECTOR)) return 4;
return value->u.caul.cElems * 4; return value->caul.cElems * 4;
case VT_I8: case VT_I8:
case VT_UI8: case VT_UI8:
case VT_R8: case VT_R8:
if (!(value->vt & VT_VECTOR)) return 8; if (!(value->vt & VT_VECTOR)) return 8;
return value->u.cauh.cElems * 8; return value->cauh.cElems * 8;
case VT_LPSTR: case VT_LPSTR:
return value->u.pszVal ? strlen(value->u.pszVal) + 1 : 0; return value->pszVal ? strlen(value->pszVal) + 1 : 0;
case VT_BLOB: case VT_BLOB:
return value->u.blob.cbSize; return value->blob.cbSize;
default: default:
FIXME("not supported variant type %d\n", value->vt); FIXME("not supported variant type %d\n", value->vt);
return 0; return 0;
@ -2545,7 +2545,7 @@ GpStatus WINGDIPAPI GdipGetPropertyItemSize(GpImage *image, PROPID propid, UINT
if (!reader) return PropertyNotFound; if (!reader) return PropertyNotFound;
id.vt = VT_UI2; id.vt = VT_UI2;
id.u.uiVal = propid; id.uiVal = propid;
hr = IWICMetadataReader_GetValue(reader, NULL, &id, &value); hr = IWICMetadataReader_GetValue(reader, NULL, &id, &value);
if (FAILED(hr)) return PropertyNotFound; if (FAILED(hr)) return PropertyNotFound;
@ -2610,38 +2610,38 @@ static GpStatus propvariant_to_item(PROPVARIANT *value, PropertyItem *item,
case VT_I1: case VT_I1:
case VT_UI1: case VT_UI1:
if (!(value->vt & VT_VECTOR)) if (!(value->vt & VT_VECTOR))
*(BYTE *)item->value = value->u.bVal; *(BYTE *)item->value = value->bVal;
else else
memcpy(item->value, value->u.caub.pElems, item_size); memcpy(item->value, value->caub.pElems, item_size);
break; break;
case VT_I2: case VT_I2:
case VT_UI2: case VT_UI2:
if (!(value->vt & VT_VECTOR)) if (!(value->vt & VT_VECTOR))
*(USHORT *)item->value = value->u.uiVal; *(USHORT *)item->value = value->uiVal;
else else
memcpy(item->value, value->u.caui.pElems, item_size); memcpy(item->value, value->caui.pElems, item_size);
break; break;
case VT_I4: case VT_I4:
case VT_UI4: case VT_UI4:
case VT_R4: case VT_R4:
if (!(value->vt & VT_VECTOR)) if (!(value->vt & VT_VECTOR))
*(ULONG *)item->value = value->u.ulVal; *(ULONG *)item->value = value->ulVal;
else else
memcpy(item->value, value->u.caul.pElems, item_size); memcpy(item->value, value->caul.pElems, item_size);
break; break;
case VT_I8: case VT_I8:
case VT_UI8: case VT_UI8:
case VT_R8: case VT_R8:
if (!(value->vt & VT_VECTOR)) if (!(value->vt & VT_VECTOR))
*(ULONGLONG *)item->value = value->u.uhVal.QuadPart; *(ULONGLONG *)item->value = value->uhVal.QuadPart;
else else
memcpy(item->value, value->u.cauh.pElems, item_size); memcpy(item->value, value->cauh.pElems, item_size);
break; break;
case VT_LPSTR: case VT_LPSTR:
memcpy(item->value, value->u.pszVal, item_size); memcpy(item->value, value->pszVal, item_size);
break; break;
case VT_BLOB: case VT_BLOB:
memcpy(item->value, value->u.blob.pBlobData, item_size); memcpy(item->value, value->blob.pBlobData, item_size);
break; break;
default: default:
FIXME("not supported variant type %d\n", value->vt); FIXME("not supported variant type %d\n", value->vt);
@ -2698,7 +2698,7 @@ GpStatus WINGDIPAPI GdipGetPropertyItem(GpImage *image, PROPID propid, UINT size
if (!reader) return PropertyNotFound; if (!reader) return PropertyNotFound;
id.vt = VT_UI2; id.vt = VT_UI2;
id.u.uiVal = propid; id.uiVal = propid;
hr = IWICMetadataReader_GetValue(reader, NULL, &id, &value); hr = IWICMetadataReader_GetValue(reader, NULL, &id, &value);
if (FAILED(hr)) return PropertyNotFound; if (FAILED(hr)) return PropertyNotFound;
@ -2851,7 +2851,7 @@ GpStatus WINGDIPAPI GdipGetAllPropertyItems(GpImage *image, UINT size,
{ {
item = heap_alloc(item_size + sizeof(*item)); item = heap_alloc(item_size + sizeof(*item));
propvariant_to_item(&value, item, item_size + sizeof(*item), id.u.uiVal); propvariant_to_item(&value, item, item_size + sizeof(*item), id.uiVal);
buf[i].id = item->id; buf[i].id = item->id;
buf[i].type = item->type; buf[i].type = item->type;
buf[i].length = item_size; buf[i].length = item_size;
@ -3040,12 +3040,12 @@ static BOOL get_bool_property(IWICMetadataReader *reader, const GUID *guid, cons
PropVariantInit(&value); PropVariantInit(&value);
id.vt = VT_LPWSTR; id.vt = VT_LPWSTR;
id.u.pwszVal = CoTaskMemAlloc((lstrlenW(prop_name) + 1) * sizeof(WCHAR)); id.pwszVal = CoTaskMemAlloc((lstrlenW(prop_name) + 1) * sizeof(WCHAR));
if (!id.u.pwszVal) return FALSE; if (!id.pwszVal) return FALSE;
lstrcpyW(id.u.pwszVal, prop_name); lstrcpyW(id.pwszVal, prop_name);
hr = IWICMetadataReader_GetValue(reader, NULL, &id, &value); hr = IWICMetadataReader_GetValue(reader, NULL, &id, &value);
if (hr == S_OK && value.vt == VT_BOOL) if (hr == S_OK && value.vt == VT_BOOL)
ret = value.u.boolVal; ret = value.boolVal;
PropVariantClear(&id); PropVariantClear(&id);
PropVariantClear(&value); PropVariantClear(&value);
@ -3067,9 +3067,9 @@ static PropertyItem *get_property(IWICMetadataReader *reader, const GUID *guid,
PropVariantInit(&value); PropVariantInit(&value);
id.vt = VT_LPWSTR; id.vt = VT_LPWSTR;
id.u.pwszVal = CoTaskMemAlloc((lstrlenW(prop_name) + 1) * sizeof(WCHAR)); id.pwszVal = CoTaskMemAlloc((lstrlenW(prop_name) + 1) * sizeof(WCHAR));
if (!id.u.pwszVal) return NULL; if (!id.pwszVal) return NULL;
lstrcpyW(id.u.pwszVal, prop_name); lstrcpyW(id.pwszVal, prop_name);
hr = IWICMetadataReader_GetValue(reader, NULL, &id, &value); hr = IWICMetadataReader_GetValue(reader, NULL, &id, &value);
if (hr == S_OK) if (hr == S_OK)
{ {
@ -3415,7 +3415,7 @@ static ULONG get_ulong_by_index(IWICMetadataReader* reader, ULONG index)
switch (value.vt) switch (value.vt)
{ {
case VT_UI4: case VT_UI4:
result = value.u.ulVal; result = value.ulVal;
break; break;
default: default:
ERR("unhandled case %u\n", value.vt); ERR("unhandled case %u\n", value.vt);
@ -3477,7 +3477,7 @@ static void png_metadata_reader(GpBitmap *bitmap, IWICBitmapDecoder *decoder, UI
if (name.vt == VT_LPSTR) if (name.vt == VT_LPSTR)
{ {
for (j = 0; j < ARRAY_SIZE(keywords); j++) for (j = 0; j < ARRAY_SIZE(keywords); j++)
if (!strcmp(keywords[j].name, name.u.pszVal)) if (!strcmp(keywords[j].name, name.pszVal))
break; break;
if (j < ARRAY_SIZE(keywords) && !keywords[j].seen) if (j < ARRAY_SIZE(keywords) && !keywords[j].seen)
{ {