GDI+: headers and one test

Francois Gouget fgouget at free.fr
Thu May 31 08:59:09 CDT 2007


On Tue, 29 May 2007, Evan Stade wrote:
[...]
> include/Gdiplus.h                |   10 +++++
> include/GdiplusEnums.h           |   39 ++++++++++++++++++
> include/GdiplusFlat.h            |   23 ++++++++++
> include/GdiplusGpStubs.h         |   10 +++++
> include/GdiplusInit.h            |   26 ++++++++++++
> include/GdiplusTypes.h           |    7 +++

All filenames in Wine should be lowercase.

--- /dev/null
+++ b/include/Gdiplus.h
[...]
+
+#ifndef __cplusplus

Why check for __cplusplus here? The PSDK contains no such check, at 
least in this file.


--- /dev/null
+++ b/include/GdiplusEnums.h
[...]
+#ifndef _GP_ENUMERATIONS_H_
+#define _GP_ENUMERATIONS_H_

The standard format for Wine headers is __WINE_HEADERNAME_H, unless we 
need to keep the original macro name for compatibility with the PSDK. 
But here you're following neither conventions (the other headers would 
need to be rechecked but they seem to keep the PSDK names).


--- /dev/null
+++ b/include/GdiplusFlat.h
[...]
+GpStatus WINGDIPAPI GdipCreateFromHDC(HDC hdc, GpGraphics **graphics);
+GpStatus WINGDIPAPI GdipDeleteGraphics(GpGraphics *graphics);

Wine headers generally don't keep the parameter names in the headers as 
they are not needed.


--- /dev/null
+++ b/include/GdiplusTypes.h
[...]
+typedef enum {
[...]
+} Status;

Hmm, this is not the same as the PSDK. The PSDK only defines 'enum 
Status { ... }' which, in theory, should only allow one to use 'enum 
Status' and not 'Status'. Yet the PSDK headers seem to make use of 
'Status'. This might be different in C++ though... 
So on the whole it's probably ok to use a typedef. I just prefer to 
bring this up so others can chime in.


-- 
Francois Gouget <fgouget at free.fr>              http://fgouget.free.fr/
                           La terre est une bêta...


More information about the wine-devel mailing list