[PATCH] gdi32: Don't bother using the return value of HeapFree().
Huw Davies
huw at codeweavers.com
Mon Jan 22 03:53:38 CST 2018
Signed-off-by: Huw Davies <huw at codeweavers.com>
---
dlls/gdi32/bitmap.c | 3 ++-
dlls/gdi32/brush.c | 3 ++-
dlls/gdi32/dib.c | 3 ++-
dlls/gdi32/dibdrv/opengl.c | 3 ++-
dlls/gdi32/enhmetafile.c | 3 ++-
dlls/gdi32/font.c | 3 ++-
dlls/gdi32/metafile.c | 4 +++-
dlls/gdi32/palette.c | 3 ++-
dlls/gdi32/pen.c | 3 ++-
9 files changed, 19 insertions(+), 9 deletions(-)
diff --git a/dlls/gdi32/bitmap.c b/dlls/gdi32/bitmap.c
index 768911a5a4..c16793d82b 100644
--- a/dlls/gdi32/bitmap.c
+++ b/dlls/gdi32/bitmap.c
@@ -489,7 +489,8 @@ static BOOL BITMAP_DeleteObject( HGDIOBJ handle )
if (!bmp) return FALSE;
HeapFree( GetProcessHeap(), 0, bmp->dib.dsBm.bmBits );
- return HeapFree( GetProcessHeap(), 0, bmp );
+ HeapFree( GetProcessHeap(), 0, bmp );
+ return TRUE;
}
diff --git a/dlls/gdi32/brush.c b/dlls/gdi32/brush.c
index 3c112439fe..7d13649305 100644
--- a/dlls/gdi32/brush.c
+++ b/dlls/gdi32/brush.c
@@ -467,7 +467,8 @@ static BOOL BRUSH_DeleteObject( HGDIOBJ handle )
if (!brush) return FALSE;
free_brush_pattern( &brush->pattern );
- return HeapFree( GetProcessHeap(), 0, brush );
+ HeapFree( GetProcessHeap(), 0, brush );
+ return TRUE;
}
diff --git a/dlls/gdi32/dib.c b/dlls/gdi32/dib.c
index 2eb0e91aa3..c7dcdb122c 100644
--- a/dlls/gdi32/dib.c
+++ b/dlls/gdi32/dib.c
@@ -1831,5 +1831,6 @@ static BOOL DIB_DeleteObject( HGDIOBJ handle )
else VirtualFree( bmp->dib.dsBm.bmBits, 0, MEM_RELEASE );
HeapFree(GetProcessHeap(), 0, bmp->color_table);
- return HeapFree( GetProcessHeap(), 0, bmp );
+ HeapFree( GetProcessHeap(), 0, bmp );
+ return TRUE;
}
diff --git a/dlls/gdi32/dibdrv/opengl.c b/dlls/gdi32/dibdrv/opengl.c
index 20cfc02398..dba5db0356 100644
--- a/dlls/gdi32/dibdrv/opengl.c
+++ b/dlls/gdi32/dibdrv/opengl.c
@@ -217,7 +217,8 @@ static struct wgl_context *dibdrv_wglCreateContext( HDC hdc )
static BOOL dibdrv_wglDeleteContext( struct wgl_context *context )
{
pOSMesaDestroyContext( context->context );
- return HeapFree( GetProcessHeap(), 0, context );
+ HeapFree( GetProcessHeap(), 0, context );
+ return TRUE;
}
/***********************************************************************
diff --git a/dlls/gdi32/enhmetafile.c b/dlls/gdi32/enhmetafile.c
index 111fa436bd..37edc231a3 100644
--- a/dlls/gdi32/enhmetafile.c
+++ b/dlls/gdi32/enhmetafile.c
@@ -287,7 +287,8 @@ static BOOL EMF_Delete_HENHMETAFILE( HENHMETAFILE hmf )
UnmapViewOfFile( metaObj->emh );
else
HeapFree( GetProcessHeap(), 0, metaObj->emh );
- return HeapFree( GetProcessHeap(), 0, metaObj );
+ HeapFree( GetProcessHeap(), 0, metaObj );
+ return TRUE;
}
/******************************************************************
diff --git a/dlls/gdi32/font.c b/dlls/gdi32/font.c
index d059f97c24..17b160d2dc 100644
--- a/dlls/gdi32/font.c
+++ b/dlls/gdi32/font.c
@@ -811,7 +811,8 @@ static BOOL FONT_DeleteObject( HGDIOBJ handle )
FONTOBJ *obj;
if (!(obj = free_gdi_handle( handle ))) return FALSE;
- return HeapFree( GetProcessHeap(), 0, obj );
+ HeapFree( GetProcessHeap(), 0, obj );
+ return TRUE;
}
diff --git a/dlls/gdi32/metafile.c b/dlls/gdi32/metafile.c
index c7e25a7fa8..28ad53cf57 100644
--- a/dlls/gdi32/metafile.c
+++ b/dlls/gdi32/metafile.c
@@ -135,8 +135,10 @@ static POINT *convert_points( UINT count, const POINTS *pts )
BOOL WINAPI DeleteMetaFile( HMETAFILE hmf )
{
METAHEADER *mh = free_gdi_handle( hmf );
+
if (!mh) return FALSE;
- return HeapFree( GetProcessHeap(), 0, mh );
+ HeapFree( GetProcessHeap(), 0, mh );
+ return TRUE;
}
/******************************************************************
diff --git a/dlls/gdi32/palette.c b/dlls/gdi32/palette.c
index a8135d5a37..52bd0d30bb 100644
--- a/dlls/gdi32/palette.c
+++ b/dlls/gdi32/palette.c
@@ -610,7 +610,8 @@ static BOOL PALETTE_DeleteObject( HGDIOBJ handle )
PALETTE_UnrealizeObject( handle );
if (!(obj = free_gdi_handle( handle ))) return FALSE;
HeapFree( GetProcessHeap(), 0, obj->entries );
- return HeapFree( GetProcessHeap(), 0, obj );
+ HeapFree( GetProcessHeap(), 0, obj );
+ return TRUE;
}
diff --git a/dlls/gdi32/pen.c b/dlls/gdi32/pen.c
index d6c1dc4576..fa789e4cf9 100644
--- a/dlls/gdi32/pen.c
+++ b/dlls/gdi32/pen.c
@@ -279,7 +279,8 @@ static BOOL PEN_DeleteObject( HGDIOBJ handle )
if (!pen) return FALSE;
free_brush_pattern( &pen->pattern );
- return HeapFree( GetProcessHeap(), 0, pen );
+ HeapFree( GetProcessHeap(), 0, pen );
+ return TRUE;
}
--
2.12.0
More information about the wine-devel
mailing list