Mike McCormack : avifil32: Eliminate some gcc 4. 1 warnings caused by windowsx.h macros.

Alexandre Julliard julliard at wine.codeweavers.com
Sat Mar 4 11:10:00 CST 2006


Module: wine
Branch: refs/heads/master
Commit: 73bc4e6752cef032406f3cf08a55cc097bc0cb23
URL:    http://source.winehq.org/git/?p=wine.git;a=commit;h=73bc4e6752cef032406f3cf08a55cc097bc0cb23

Author: Mike McCormack <mike at codeweavers.com>
Date:   Sat Mar  4 14:13:05 2006 +0900

avifil32: Eliminate some gcc 4.1 warnings caused by windowsx.h macros.

---

 dlls/avifil32/acmstream.c |   41 ++++++++++++++++++++++++++---------------
 1 files changed, 26 insertions(+), 15 deletions(-)

diff --git a/dlls/avifil32/acmstream.c b/dlls/avifil32/acmstream.c
index 0dc068f..6ffe416 100644
--- a/dlls/avifil32/acmstream.c
+++ b/dlls/avifil32/acmstream.c
@@ -26,7 +26,6 @@
 #include "winuser.h"
 #include "winnls.h"
 #include "winerror.h"
-#include "windowsx.h"
 #include "mmsystem.h"
 #include "vfw.h"
 #include "msacm.h"
@@ -177,20 +176,24 @@ static ULONG WINAPI ACMStream_fnRelease(
       This->has = NULL;
     }
     if (This->acmStreamHdr.pbSrc != NULL) {
-      GlobalFreePtr(This->acmStreamHdr.pbSrc);
+      GlobalUnlock(GlobalHandle(This->acmStreamHdr.pbSrc));
+      GlobalFree(GlobalHandle(This->acmStreamHdr.pbSrc));
       This->acmStreamHdr.pbSrc = NULL;
     }
     if (This->acmStreamHdr.pbDst != NULL) {
-      GlobalFreePtr(This->acmStreamHdr.pbDst);
+      GlobalUnlock(GlobalHandle(This->acmStreamHdr.pbDst));
+      GlobalFree(GlobalHandle(This->acmStreamHdr.pbDst));
       This->acmStreamHdr.pbDst = NULL;
     }
     if (This->lpInFormat != NULL) {
-      GlobalFreePtr(This->lpInFormat);
+      GlobalUnlock(GlobalHandle(This->lpInFormat));
+      GlobalFree(GlobalHandle(This->lpInFormat));
       This->lpInFormat = NULL;
       This->cbInFormat = 0;
     }
     if (This->lpOutFormat != NULL) {
-      GlobalFreePtr(This->lpOutFormat);
+      GlobalUnlock(GlobalHandle(This->lpOutFormat));
+      GlobalFree(GlobalHandle(This->lpOutFormat));
       This->lpOutFormat = NULL;
       This->cbOutFormat = 0;
     }
@@ -250,7 +253,7 @@ static HRESULT WINAPI ACMStream_fnCreate
     else
       This->cbOutFormat = sizeof(PCMWAVEFORMAT);
 
-    This->lpOutFormat = (LPWAVEFORMATEX)GlobalAllocPtr(GHND, This->cbOutFormat);
+    This->lpOutFormat = GlobalLock(GlobalAlloc(GHND, This->cbOutFormat));
     if (This->lpOutFormat == NULL)
       return AVIERR_MEMORY;
 
@@ -383,7 +386,7 @@ static HRESULT WINAPI ACMStream_fnSetFor
   if ((This->sInfo.dwCaps & AVIFILECAPS_CANWRITE) == 0)
     return AVIERR_READONLY;
 
-  This->lpInFormat = (LPWAVEFORMATEX)GlobalAllocPtr(GMEM_MOVEABLE, formatsize);
+  This->lpInFormat = GlobalLock(GlobalAlloc(GMEM_MOVEABLE, formatsize));
   if (This->lpInFormat == NULL)
     return AVIERR_MEMORY;
   This->cbInFormat = formatsize;
@@ -463,7 +466,8 @@ static HRESULT WINAPI ACMStream_fnRead(I
 
   /* Need to free destination buffer used for writing? */
   if (This->acmStreamHdr.pbDst != NULL) {
-    GlobalFreePtr(This->acmStreamHdr.pbDst);
+    GlobalUnlock(GlobalHandle(This->acmStreamHdr.pbDst));
+    GlobalFree(GlobalHandle(This->acmStreamHdr.pbDst));
     This->acmStreamHdr.pbDst     = NULL;
     This->acmStreamHdr.dwDstUser = 0;
   }
@@ -472,10 +476,13 @@ static HRESULT WINAPI ACMStream_fnRead(I
   if (This->acmStreamHdr.pbSrc == NULL ||
       This->acmStreamHdr.dwSrcUser < size) {
     if (This->acmStreamHdr.pbSrc == NULL)
-      This->acmStreamHdr.pbSrc = GlobalAllocPtr(GMEM_MOVEABLE, size);
+      This->acmStreamHdr.pbSrc = GlobalLock(GlobalAlloc(GMEM_MOVEABLE, size));
     else
-      This->acmStreamHdr.pbSrc = GlobalReAllocPtr(This->acmStreamHdr.pbSrc,
+    {
+      GlobalUnlock(GlobalHandle(This->acmStreamHdr.pbSrc));
+      This->acmStreamHdr.pbDst = GlobalReAlloc(GlobalHandle(This->acmStreamHdr.pbSrc),
 						  size, GMEM_MOVEABLE);
+    }
     if (This->acmStreamHdr.pbSrc == NULL)
       return AVIERR_MEMORY;
     This->acmStreamHdr.dwSrcUser = size;
@@ -567,7 +574,8 @@ static HRESULT WINAPI ACMStream_fnWrite(
 
   /* Need to free source buffer used for reading? */
   if (This->acmStreamHdr.pbSrc != NULL) {
-    GlobalFreePtr(This->acmStreamHdr.pbSrc);
+    GlobalUnlock(GlobalHandle(This->acmStreamHdr.pbSrc));
+    GlobalFree(GlobalHandle(This->acmStreamHdr.pbSrc));
     This->acmStreamHdr.pbSrc     = NULL;
     This->acmStreamHdr.dwSrcUser = 0;
   }
@@ -576,10 +584,13 @@ static HRESULT WINAPI ACMStream_fnWrite(
   if (This->acmStreamHdr.pbDst == NULL ||
       This->acmStreamHdr.dwDstUser < size) {
     if (This->acmStreamHdr.pbDst == NULL)
-      This->acmStreamHdr.pbDst = GlobalAllocPtr(GMEM_MOVEABLE, size);
+      This->acmStreamHdr.pbDst = GlobalLock(GlobalAlloc(GMEM_MOVEABLE, size));
     else
-      This->acmStreamHdr.pbDst = GlobalReAllocPtr(This->acmStreamHdr.pbDst,
+    {
+      GlobalUnlock(GlobalHandle(This->acmStreamHdr.pbDst));
+      This->acmStreamHdr.pbDst = GlobalReAlloc(GlobalHandle(This->acmStreamHdr.pbDst),
 						  size, GMEM_MOVEABLE);
+    }
     if (This->acmStreamHdr.pbDst == NULL)
       return AVIERR_MEMORY;
     This->acmStreamHdr.dwDstUser = size;
@@ -703,7 +714,7 @@ static HRESULT AVIFILE_OpenCompressor(IA
     hr = AVIStreamFormatSize(This->pStream, This->sInfo.dwStart, &This->cbInFormat);
     if (FAILED(hr))
       return hr;
-    This->lpInFormat = (LPWAVEFORMATEX)GlobalAllocPtr(GMEM_MOVEABLE, This->cbInFormat);
+    This->lpInFormat = GlobalLock(GlobalAlloc(GMEM_MOVEABLE, This->cbInFormat));
     if (This->lpInFormat == NULL)
       return AVIERR_MEMORY;
 
@@ -715,7 +726,7 @@ static HRESULT AVIFILE_OpenCompressor(IA
     if (This->lpOutFormat == NULL) {
       /* we must decode to default format */
       This->cbOutFormat = sizeof(PCMWAVEFORMAT);
-      This->lpOutFormat = (LPWAVEFORMATEX)GlobalAllocPtr(GHND, This->cbOutFormat);
+      This->lpOutFormat = GlobalLock(GlobalAlloc(GHND, This->cbOutFormat));
       if (This->lpOutFormat == NULL)
 	return AVIERR_MEMORY;
 




More information about the wine-cvs mailing list