make comm.c verbose

Andreas Mohr andi at rhlx01.fht-esslingen.de
Sun May 19 16:23:11 CDT 2002


Hi all,

added FIXME messages in case of missing OS comm support

-- 
Andreas Mohr                        Stauferstr. 6, D-71272 Renningen, Germany
-------------- next part --------------
Determining best CVS host...
Using CVSROOT :pserver:cvs at rhlx01.fht-esslingen.de:/home/wine
Index: dlls/kernel/comm.c
===================================================================
RCS file: /home/wine/wine/dlls/kernel/comm.c,v
retrieving revision 1.60
diff -u -r1.60 comm.c
--- dlls/kernel/comm.c	14 May 2002 23:18:24 -0000	1.60
+++ dlls/kernel/comm.c	19 May 2002 21:22:27 -0000
@@ -608,32 +608,40 @@
 #ifdef TIOCM_DTR
 			direct=TRUE;
 			result= COMM_WhackModem(fd, ~TIOCM_DTR, 0);
-			break;
+#else
+			FIXME("no OS support for CLRDTR !\n");
 #endif
+			break;
 
 		case CLRRTS:
 		        TRACE("CLRRTS\n");
 #ifdef TIOCM_RTS
 			direct=TRUE;
 			result= COMM_WhackModem(fd, ~TIOCM_RTS, 0);
-			break;
+#else
+			FIXME("no OS support for CLRRTS !\n");
 #endif
+			break;
 	
 		case SETDTR:
 		        TRACE("SETDTR\n");
 #ifdef TIOCM_DTR
 			direct=TRUE;
 			result= COMM_WhackModem(fd, 0, TIOCM_DTR);
-			break;
+#else
+			FIXME("no OS support for SETDTR !\n");
 #endif
+			break;
 
 		case SETRTS:
 		        TRACE("SETRTS\n");
 #ifdef TIOCM_RTS
 			direct=TRUE;
 			result= COMM_WhackModem(fd, 0, TIOCM_RTS);
-			break;
+#else
+			FIXME("no OS support for SETRTS !\n");
 #endif
+			break;
 
 		case SETXOFF:
 		        TRACE("SETXOFF\n");
@@ -649,15 +657,19 @@
 #ifdef 	TIOCSBRK
 			direct=TRUE;
 			result = ioctl(fd,TIOCSBRK,0);
-			break;
+#else
+			FIXME("no OS support for setbreak !\n");
 #endif
+			break;
 		case CLRBREAK:
 			TRACE("clrbreak\n");
 #ifdef 	TIOCSBRK
 			direct=TRUE;
 			result = ioctl(fd,TIOCCBRK,0);
-			break;
+#else
+			FIXME("no OS support for clrbreak !\n");
 #endif
+			break;
 		default:
 			WARN("(handle=%d,nFunction=%d): Unknown function\n", 
 			handle, nFunction);
@@ -759,12 +771,15 @@
 	if(ioctl(fd, TIOCOUTQ, &lpStat->cbOutQue))
 	    WARN("ioctl returned error\n");
 #else
+	FIXME("no OS support for TIOCOUTQ ioctl !\n");
 	lpStat->cbOutQue = 0; /* FIXME: find a different way to find out */
 #endif
 
 #ifdef TIOCINQ
 	if(ioctl(fd, TIOCINQ, &lpStat->cbInQue))
 	    WARN("ioctl returned error\n");
+#else
+	FIXME("no OS support for TIOCINQ ioctl !\n");
 #endif
 
 	TRACE("handle %d cbInQue = %ld cbOutQue = %ld\n",
@@ -1205,7 +1220,7 @@
 	else
 		port.c_iflag &= ~IXOFF;
 
-	if (tcsetattr(fd,TCSANOW,&port)==-1) { /* otherwise it hangs with pending input*/
+	if (tcsetattr(fd,TCSANOW,&port)==-1) { /* otherwise it hangs with pending input */
 	        int save_error=errno;
                 COMM_SetCommError(handle,CE_IOE);
                 close( fd );
@@ -1726,7 +1741,7 @@
  *   FALSE if failure
  *
  *   The set of detected events will be written to *lpdwEventMask
- *   ERROR_IO_PENDING will be returned the overlapped structure was passed
+ *   ERROR_IO_PENDING will be returned if the OVERLAPPED structure was passed
  *
  * BUGS:
  *  Only supports EV_RXCHAR and EV_TXEMPTY
@@ -1739,7 +1754,7 @@
     OVERLAPPED ov;
     int ret;
 
-    TRACE("(%x %p %p )\n",hFile, lpdwEvents,lpOverlapped);
+    TRACE("(%x %p %p)\n",hFile, lpdwEvents,lpOverlapped);
 
     if(lpOverlapped)
         return COMM_WaitCommEvent(hFile, lpdwEvents, lpOverlapped);


More information about the wine-patches mailing list