From 9650c2baea2d4448047ae60d3209a796692ddace Mon Sep 17 00:00:00 2001
From: Andreas Mohr <a.mohr@mailto.de>
Date: Thu, 24 May 2001 18:42:38 +0000
Subject: [PATCH] - BuildCommDCB16() uses int instead of BOOL - OpenComm16()
 should return IE_BADID for *all* invalid strings

---
 dlls/kernel/comm.c       | 4 ++--
 include/wine/winuser16.h | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/dlls/kernel/comm.c b/dlls/kernel/comm.c
index 7b7bad449b3..935a27b5235 100644
--- a/dlls/kernel/comm.c
+++ b/dlls/kernel/comm.c
@@ -379,7 +379,7 @@ static void comm_waitwrite(struct DosDeviceStruct *ptr)
  * IF THIS IS NOT CORRECT THE RETURNVALUE CHECK IN BuildCommDCBAndTimeoutsA
  * NEEDS TO BE FIXED
  */
-BOOL16 WINAPI BuildCommDCB16(LPCSTR device, LPDCB16 lpdcb)
+INT16 WINAPI BuildCommDCB16(LPCSTR device, LPDCB16 lpdcb)
 {
 	/* "COM1:96,n,8,1"	*/
 	/*  012345		*/
@@ -600,7 +600,7 @@ INT16 WINAPI OpenComm16(LPCSTR device,UINT16 cbInQueue,UINT16 cbOutQueue)
 			return port|FLAG_LPT;
 		}
 	}
-	return 0;
+	return IE_BADID;
 }
 
 /*****************************************************************************
diff --git a/include/wine/winuser16.h b/include/wine/winuser16.h
index 9b53e3b4065..c48697568e2 100644
--- a/include/wine/winuser16.h
+++ b/include/wine/winuser16.h
@@ -920,7 +920,7 @@ INT16       WINAPI wvsprintf16(LPSTR,LPCSTR,LPCVOID);
 BOOL16      WINAPI DrawState16A(HDC16,HBRUSH16,DRAWSTATEPROC16,LPARAM,WPARAM16,INT16,INT16,INT16,INT16,UINT16);
 BOOL16      WINAPI IsDialogMessage16(HWND16,SEGPTR);
 INT16       WINAPI GetCommError16(INT16,LPCOMSTAT16);
-BOOL16      WINAPI BuildCommDCB16(LPCSTR,LPDCB16);
+INT16       WINAPI BuildCommDCB16(LPCSTR,LPDCB16);
 INT16       WINAPI GetCommState16(INT16,LPDCB16);
 INT16       WINAPI SetCommState16(LPDCB16);
 INT16       WINAPI lstrcmp16(LPCSTR,LPCSTR);