[PATCH 0/1] MR9117: gdi32: Add stub for GetEnhMetaFilePixelFormat.

[Louis Lenders (＠xe)]

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=58747

[PATCH 1/1] gdi32: Add stub for GetEnhMetaFilePixelFormat.

[Louis Lenders]

From: Louis Lenders

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=58747
---
 dlls/gdi32/enhmetafile.c | 9 +++++++++
 dlls/gdi32/gdi32.spec    | 2 +-
 include/wingdi.h         | 1 +
 3 files changed, 11 insertions(+), 1 deletion(-)

diff --git a/dlls/gdi32/enhmetafile.c b/dlls/gdi32/enhmetafile.c
index 89622ef1b58..07d464fe147 100644
--- a/dlls/gdi32/enhmetafile.c
+++ b/dlls/gdi32/enhmetafile.c
@@ -3001,3 +3001,12 @@ end:
     DeleteMetaFile(hmf);
     return ret;
 }
+
+/******************************************************************
+ *         GetEnhMetaFilePixelFormat   (GDI32.@)
+ */
+UINT WINAPI GetEnhMetaFilePixelFormat( HENHMETAFILE hemf, UINT size, PIXELFORMATDESCRIPTOR *ppfd )
+{
+    FIXME( "(%p,%d,%p)\n", hemf, size, ppfd );
+    return 0;
+}
diff --git a/dlls/gdi32/gdi32.spec b/dlls/gdi32/gdi32.spec
index ae097e607ba..f51931f530a 100644
--- a/dlls/gdi32/gdi32.spec
+++ b/dlls/gdi32/gdi32.spec
@@ -375,7 +375,7 @@
 @ stdcall GetEnhMetaFileDescriptionW(long long ptr)
 @ stdcall GetEnhMetaFileHeader(long long ptr)
 @ stdcall GetEnhMetaFilePaletteEntries(long long ptr)
-# @ stub GetEnhMetaFilePixelFormat
+@ stdcall GetEnhMetaFilePixelFormat(ptr long ptr)
 @ stdcall GetEnhMetaFileW(wstr)
 # @ stub GetFontAssocStatus
 @ stdcall GetFontData(long long long ptr long) NtGdiGetFontData
diff --git a/include/wingdi.h b/include/wingdi.h
index ed0e59462e9..559f659297f 100644
--- a/include/wingdi.h
+++ b/include/wingdi.h
@@ -3957,6 +3957,7 @@ WINGDIAPI UINT        WINAPI GetEnhMetaFileDescriptionW(HENHMETAFILE,UINT,LPWSTR
 #define                      GetEnhMetaFileDescription WINELIB_NAME_AW(GetEnhMetaFileDescription)
 WINGDIAPI UINT        WINAPI GetEnhMetaFileHeader(HENHMETAFILE,UINT,LPENHMETAHEADER);
 WINGDIAPI UINT        WINAPI GetEnhMetaFilePaletteEntries(HENHMETAFILE,UINT,LPPALETTEENTRY);
+WINGDIAPI UINT        WINAPI GetEnhMetaFilePixelFormat(HENHMETAFILE,UINT,PIXELFORMATDESCRIPTOR*);
 WINGDIAPI DWORD       WINAPI GetFontData(HDC,DWORD,DWORD,LPVOID,DWORD);
 WINGDIAPI DWORD       WINAPI GetFontLanguageInfo(HDC);
 WINGDIAPI DWORD       WINAPI GetFontUnicodeRanges(HDC,LPGLYPHSET);

[Dmitry Timoshkov (＠dmitry)]

Dmitry Timoshkov (@dmitry) commented about dlls/gdi32/enhmetafile.c:
>  DeleteMetaFile(hmf);
>  return ret;
>
> }
> -
>
> +/******************************************************************
> -
> -
>     GetEnhMetaFilePixelFormat   (GDI32.@)
> - */
>
> +UINT WINAPI GetEnhMetaFilePixelFormat( HENHMETAFILE hemf, UINT size, PIXELFORMATDESCRIPTOR *ppfd )
> +{
> - FIXME( "(%p,%d,%p)\n", hemf, size, ppfd );
> - return 0;

Probably it should return GDI_ERROR. Also, usually ": stub" is appened in the FIXME message for stubs.

[Dmitry Timoshkov (＠dmitry)]

On Tue Oct  7 11:29:30 2025 +0000, Dmitry Timoshkov wrote:
> Probably it should return GDI_ERROR. Also, usually ": stub" is appened
> in the FIXME message for stubs.

In addition, "%u" should be used for an UINT argument.