From bca4a8dce0b2885fbbf54ae3d3759ab33c3a16cb Mon Sep 17 00:00:00 2001
From: Patrik Stridvall <ps@leissner.se>
Date: Mon, 12 Feb 2001 03:49:57 +0000
Subject: [PATCH] Documentation argument fixes.

---
 dlls/kernel/comm.c       | 163 ++++++++++++++++++++-------------------
 dlls/kernel/debugger.c   |  20 ++---
 dlls/kernel/time.c       |  33 ++++----
 dlls/oleaut32/dispatch.c |  45 +++++------
 files/file.c             |  22 +++---
 5 files changed, 145 insertions(+), 138 deletions(-)

diff --git a/dlls/kernel/comm.c b/dlls/kernel/comm.c
index 6bfa76203af..76606e5ceca 100644
--- a/dlls/kernel/comm.c
+++ b/dlls/kernel/comm.c
@@ -1499,8 +1499,8 @@ BOOL16 WINAPI EnableCommNotification16( INT16 cid, HWND16 hwnd,
  *  True on success, false on an malformed control string.
  */
 BOOL WINAPI BuildCommDCBA(
-	LPCSTR device, /* The ascii device control string used to update the DCB. */
-	LPDCB lpdcb /* The device control block to be updated. */)
+    LPCSTR device, /* [in] The ascii device control string used to update the DCB. */
+    LPDCB  lpdcb)  /* [out] The device control block to be updated. */
 {
 	return BuildCommDCBAndTimeoutsA(device,lpdcb,NULL);
 }
@@ -1517,9 +1517,9 @@ BOOL WINAPI BuildCommDCBA(
  *  True on success, false bad handles etc
  */
 BOOL WINAPI BuildCommDCBAndTimeoutsA(
-	LPCSTR device, /* The ascii device control string. */
-	LPDCB lpdcb, /* The device control block to be updated. */
-    LPCOMMTIMEOUTS lptimeouts /* The time outs to use if asked to set them by the control string. */)
+    LPCSTR         device,     /* [in] The ascii device control string. */
+    LPDCB          lpdcb,      /* [out] The device control block to be updated. */
+    LPCOMMTIMEOUTS lptimeouts) /* [in] The time outs to use if asked to set them by the control string. */
 {
 	int	port;
 	char	*ptr,*temp;
@@ -1651,9 +1651,9 @@ BOOL WINAPI BuildCommDCBAndTimeoutsA(
  *  True on success, false bad handles etc.
  */
 BOOL WINAPI BuildCommDCBAndTimeoutsW(
-	LPCWSTR devid, /* The unicode device control string. */
-	LPDCB lpdcb, /* The device control block to be updated. */
-    LPCOMMTIMEOUTS lptimeouts /* The time outs to use if asked to set them by the control string. */)
+    LPCWSTR        devid,      /* [in] The unicode device control string. */
+    LPDCB          lpdcb,      /* [out] The device control block to be updated. */
+    LPCOMMTIMEOUTS lptimeouts) /* [in] The time outs to use if asked to set them by the control string. */
 {
 	BOOL ret = FALSE;
 	LPSTR	devidA;
@@ -1680,8 +1680,8 @@ BOOL WINAPI BuildCommDCBAndTimeoutsW(
  *  True on success, false on an malformed control string.
  */
 BOOL WINAPI BuildCommDCBW(
-	LPCWSTR devid, /* The unicode device control string. */
-	LPDCB lpdcb /* The device control block to be updated. */)
+    LPCWSTR devid, /* [in] The unicode device control string. */
+    LPDCB   lpdcb) /* [out] The device control block to be updated. */
 {
 	return BuildCommDCBAndTimeoutsW(devid,lpdcb,NULL);
 }
@@ -1704,7 +1704,7 @@ int commerror=0;
  *  Only TIOCSBRK and TIOCCBRK are supported. 
  */
 BOOL WINAPI SetCommBreak(
-		HANDLE handle /* The communictions device to suspend. */)
+    HANDLE handle) /* [in] The communictions device to suspend. */
 {
 #if defined(TIOCSBRK) && defined(TIOCCBRK) /* check if available for compilation */
         int fd,result;
@@ -1744,7 +1744,7 @@ BOOL WINAPI SetCommBreak(
  *  Only TIOCSBRK and TIOCCBRK are supported. 
  */
 BOOL WINAPI ClearCommBreak(
-		HANDLE handle /* The halted communication device whose character transmission is to be resumed. */)
+    HANDLE handle) /* [in] The halted communication device whose character transmission is to be resumed. */
 {
 #if defined(TIOCSBRK) && defined(TIOCCBRK) /* check if available for compilation */
         int fd,result;
@@ -1782,8 +1782,8 @@ BOOL WINAPI ClearCommBreak(
  *  or the command failed.
  */
 BOOL WINAPI EscapeCommFunction(
-		HANDLE handle, /* The communication device to perform the extended function. */
-		UINT nFunction /* The extended function to be performed. */)
+    HANDLE handle,    /* [in] The communication device to perform the extended function. */
+    UINT   nFunction) /* [in] The extended function to be performed. */
 {
 	int fd,direct=FALSE,result=FALSE;
 	struct termios	port;
@@ -1899,8 +1899,8 @@ BOOL WINAPI EscapeCommFunction(
  *  True on success and false if the communications handle is bad.
  */
 BOOL WINAPI PurgeComm(
-		HANDLE handle, /* The communication resource to be purged. */
-		DWORD flags  /* Flags for clear pending/buffer on input/output. */) 
+    HANDLE handle, /* [in] The communication resource to be purged. */
+    DWORD  flags)  /* [in] Flags for clear pending/buffer on input/output. */
 {
      int fd;
 
@@ -1941,9 +1941,9 @@ BOOL WINAPI PurgeComm(
  *  True on success, false if the communication resource handle is bad.
  */
 BOOL WINAPI ClearCommError(
-		HANDLE handle, /* The communication resource with the error. */
-		LPDWORD errors, /* Flags indicating error the resource experienced. */
-		LPCOMSTAT lpStat /* The status of the communication resource. */)
+    HANDLE    handle, /* [in] The communication resource with the error. */
+    LPDWORD   errors, /* [out] Flags indicating error the resource experienced. */
+    LPCOMSTAT lpStat) /* [out] The status of the communication resource. */
 {
     int fd;
 
@@ -2004,9 +2004,9 @@ BOOL WINAPI ClearCommError(
  *  Stub.
  */
 BOOL WINAPI SetupComm(
-		HANDLE handle, /* The just created communication resource handle. */
-		DWORD insize, /* The suggested size of the communication resources input buffer in bytes. */
-		DWORD outsize /* The suggested size of the communication resources output buffer in bytes. */)
+    HANDLE handle,  /* [in] The just created communication resource handle. */
+    DWORD  insize,  /* [in] The suggested size of the communication resources input buffer in bytes. */
+    DWORD  outsize) /* [in] The suggested size of the communication resources output buffer in bytes. */
 {
     int fd;
 
@@ -2031,8 +2031,8 @@ BOOL WINAPI SetupComm(
  *   True on success, fail on bad device handle etc.
  */
 BOOL WINAPI GetCommMask(
-    HANDLE handle, /* The communications device. */
-    LPDWORD evtmask /* The events which cause WaitCommEvent to return. */)
+    HANDLE  handle,  /* [in] The communications device. */
+    LPDWORD evtmask) /* [out] The events which cause WaitCommEvent to return. */
 {
     BOOL ret;
 
@@ -2063,8 +2063,8 @@ BOOL WINAPI GetCommMask(
  *  True on success, false on bad handle etc.
  */
 BOOL WINAPI SetCommMask(
-    HANDLE handle, /* The communications device.  */
-    DWORD evtmask /* The events that to be monitored. */)
+    HANDLE handle,  /* [in] The communications device.  */
+    DWORD  evtmask) /* [in] The events that to be monitored. */
 {
     BOOL ret;
 
@@ -2094,8 +2094,8 @@ BOOL WINAPI SetCommMask(
  *  True on success, false on failure eg if the XonChar is equal to the XoffChar.
  */
 BOOL WINAPI SetCommState(
-	HANDLE handle, /* The communications device. */
-	LPDCB lpdcb /* The device control block. */)
+    HANDLE handle, /* [in] The communications device. */
+    LPDCB  lpdcb)  /* [out] The device control block. */
 {
      struct termios port;
      int fd, bytesize, stopbits;
@@ -2406,8 +2406,8 @@ BOOL WINAPI SetCommState(
  *  XonChar and XoffChar are not set.
  */
 BOOL WINAPI GetCommState(
-	HANDLE handle, /* The communications device. */
-	LPDCB lpdcb /* The device control block. */)
+    HANDLE handle, /* [in] The communications device. */
+    LPDCB  lpdcb)  /* [out] The device control block. */
 {
      struct termios port;
      int fd,speed;
@@ -2612,8 +2612,8 @@ BOOL WINAPI GetCommState(
  *  Stub.
  */
 BOOL WINAPI TransmitCommChar(
-	HANDLE hComm, /* The communication device in need of a command character. */
-	CHAR chTransmit /* The character to transmit. */)
+    HANDLE hComm,      /* [in] The communication device in need of a command character. */
+    CHAR   chTransmit) /* [in] The character to transmit. */
 {
     	FIXME("(%x,'%c'), use win32 handle!\n",hComm,chTransmit);
 	return TRUE;
@@ -2630,8 +2630,8 @@ BOOL WINAPI TransmitCommChar(
  *  or the target structure is null.
  */
 BOOL WINAPI GetCommTimeouts(
-	HANDLE hComm, /* The communications device. */
-	LPCOMMTIMEOUTS lptimeouts /* The struct of request time outs. */)
+    HANDLE         hComm,      /* [in] The communications device. */
+    LPCOMMTIMEOUTS lptimeouts) /* [out] The struct of request time outs. */
 {
     BOOL ret;
 
@@ -2677,9 +2677,9 @@ BOOL WINAPI GetCommTimeouts(
  *  True if the time outs were set, false otherwise.
  */
 BOOL WINAPI SetCommTimeouts(
-    HANDLE hComm,             /* [in] handle of COMM device */
-    LPCOMMTIMEOUTS lptimeouts /* [in] pointer to COMMTIMEOUTS structure */
-) {
+    HANDLE hComm,              /* [in] handle of COMM device */
+    LPCOMMTIMEOUTS lptimeouts) /* [in] pointer to COMMTIMEOUTS structure */
+{
     BOOL ret;
     int fd;
     struct termios tios;
@@ -2739,8 +2739,8 @@ BOOL WINAPI SetCommTimeouts(
  *  control register access, false otherwise.
  */
 BOOL WINAPI GetCommModemStatus(
-	HANDLE hFile, /* The communications device. */
-	LPDWORD lpModemStat /* The control register bits. */)
+    HANDLE  hFile,       /* [in] The communications device. */
+    LPDWORD lpModemStat) /* [out] The control register bits. */
 {
 	int fd,mstat, result=FALSE;
 	
@@ -2817,10 +2817,10 @@ VOID COMM_WaitCommEventService(void **args)
  *  Only supports EV_RXCHAR and EV_TXEMPTY
  */
 BOOL WINAPI WaitCommEvent(
-    HANDLE hFile,             /* [in] handle of comm port to wait for */
-    LPDWORD lpdwEvents,       /* [out] event(s) that were detected */
-    LPOVERLAPPED lpOverlapped /* [in/out] for Asynchronous waiting */
-) {
+    HANDLE hFile,              /* [in] handle of comm port to wait for */
+    LPDWORD lpdwEvents,        /* [out] event(s) that were detected */
+    LPOVERLAPPED lpOverlapped) /* [in/out] for Asynchronous waiting */
+{
     OVERLAPPED ov;
     LPOVERLAPPED lpov;
     int ret;
@@ -2910,9 +2910,9 @@ BOOL WINAPI WaitCommEvent(
  *  properties of the comm port.
  */
 BOOL WINAPI GetCommProperties(
-	HANDLE hFile,         /* [in] handle of the comm port */
-	LPCOMMPROP lpCommProp /* [out] pointer to struct to be filled */
-) {
+    HANDLE hFile,          /* [in] handle of the comm port */
+    LPCOMMPROP lpCommProp) /* [out] pointer to struct to be filled */
+{
     FIXME("(%d %p )\n",hFile,lpCommProp);
     if(!lpCommProp)
         return FALSE;
@@ -2980,10 +2980,10 @@ static CHAR lpszSerialUI[] = "serialui.dll";
  * Perhaps this should be done when the comm port is closed?
  */
 BOOL WINAPI CommConfigDialogA(
-	LPCSTR lpszDevice,         /* [in] name of communications device */
-	HANDLE hWnd,               /* [in] parent window for the dialog */
-	LPCOMMCONFIG lpCommConfig  /* [out] pointer to struct to fill */
-) {
+    LPCSTR lpszDevice,         /* [in] name of communications device */
+    HANDLE hWnd,               /* [in] parent window for the dialog */
+    LPCOMMCONFIG lpCommConfig) /* [out] pointer to struct to fill */
+{
     FARPROC lpfnCommDialog;
     HMODULE hConfigModule;
     BOOL r;
@@ -3012,10 +3012,10 @@ BOOL WINAPI CommConfigDialogA(
  * see CommConfigDialogA for more info
  */
 BOOL WINAPI CommConfigDialogW(
-	LPCWSTR lpszDevice,        /* [in] name of communications device */
-	HANDLE hWnd,               /* [in] parent window for the dialog */
-	LPCOMMCONFIG lpCommConfig  /* [out] pointer to struct to fill */
-) {
+    LPCWSTR lpszDevice,        /* [in] name of communications device */
+    HANDLE hWnd,               /* [in] parent window for the dialog */
+    LPCOMMCONFIG lpCommConfig) /* [out] pointer to struct to fill */
+{
     BOOL r;
     LPSTR lpDeviceA;
 
@@ -3044,11 +3044,14 @@ BOOL WINAPI CommConfigDialogW(
  *  BOOL WINAPI GetCommConfig(HANDLE hFile,LPCOMMCONFIG lpCommConfig,LPDWORD lpdwSize)
  */
 BOOL WINAPI GetCommConfig(
-	HANDLE hFile, /* The communications device. */
-	LPCOMMCONFIG lpCommConfig /* The communications configuration of the device (if it fits). */
-	/* LPDWORD lpdwSize Initially the size of the configuration buffer/structure,
-					    afterwards the number of bytes copied to the buffer or the needed size of the buffer. */
-) {
+    HANDLE       hFile,        /* [in] The communications device. */
+    LPCOMMCONFIG lpCommConfig) /* [out] The communications configuration of the device (if it fits). */
+#if 0 /* FIXME: Why is this "commented" out? */
+    LPDWORD      lpdwSize)     /* [in/out] Initially the size of the configuration buffer/structure,
+                                  afterwards the number of bytes copied to the buffer or 
+                                  the needed size of the buffer. */
+#endif
+{
     BOOL r;
 
     TRACE("(%x %p)\n",hFile,lpCommConfig);
@@ -3077,9 +3080,9 @@ BOOL WINAPI GetCommConfig(
  *  True on success, false if the handle was bad is not a communications device.
  */
 BOOL WINAPI SetCommConfig(
-	HANDLE hFile, /* The communications device. */
-	LPCOMMCONFIG lpCommConfig /* The desired configuration. */
-) {
+    HANDLE       hFile,        /* [in] The communications device. */
+    LPCOMMCONFIG lpCommConfig) /* [in] The desired configuration. */
+{
     TRACE("(%x %p)\n",hFile,lpCommConfig);
     return SetCommState(hFile,&lpCommConfig->dcb);
 }
@@ -3095,10 +3098,10 @@ BOOL WINAPI SetCommConfig(
  *  True if the device was found and the defaults set, false otherwise
  */
 BOOL WINAPI SetDefaultCommConfigA(
-	LPCSTR lpszDevice, /* The ascii name of the device targeted for configuration. */
-	LPCOMMCONFIG lpCommConfig, /* The default configuration for the device. */
-	DWORD dwSize /* The number of bytes in the configuration structure. */
-) {
+    LPCSTR       lpszDevice,   /* [in] The ascii name of the device targeted for configuration. */
+    LPCOMMCONFIG lpCommConfig, /* [in] The default configuration for the device. */
+    DWORD        dwSize)       /* [in] The number of bytes in the configuration structure. */
+{
     FARPROC lpfnSetDefaultCommConfig;
     HMODULE hConfigModule;
     BOOL r;
@@ -3132,10 +3135,10 @@ BOOL WINAPI SetDefaultCommConfigA(
  *
  */
 BOOL WINAPI SetDefaultCommConfigW(
-	LPCWSTR lpszDevice, /* The unicode name of the device targeted for configuration. */
-	LPCOMMCONFIG lpCommConfig, /* The default configuration for the device. */
-	DWORD dwSize /* The number of bytes in the configuration structure. */
-) {
+    LPCWSTR      lpszDevice,   /* [in] The unicode name of the device targeted for configuration. */
+    LPCOMMCONFIG lpCommConfig, /* [in] The default configuration for the device. */
+    DWORD        dwSize)       /* [in] The number of bytes in the configuration structure. */
+{
     BOOL r;
     LPSTR lpDeviceA;
 
@@ -3161,10 +3164,11 @@ BOOL WINAPI SetDefaultCommConfigW(
  *   if the device is not found or the buffer is too small.
  */
 BOOL WINAPI GetDefaultCommConfigA(
-	LPCSTR lpszName, /* The ascii name of the device targeted for configuration. */
-	LPCOMMCONFIG lpCC, /* The default configuration for the device. */
-    LPDWORD lpdwSize ) /* Initially the size of the default configuration buffer,
-                        afterwards the number of bytes copied to the buffer or the needed size of the buffer. */
+    LPCSTR       lpszName, /* [in] The ascii name of the device targeted for configuration. */
+    LPCOMMCONFIG lpCC,     /* [out] The default configuration for the device. */
+    LPDWORD      lpdwSize) /* [in/out] Initially the size of the default configuration buffer,
+                              afterwards the number of bytes copied to the buffer or
+                              the needed size of the buffer. */
 {
      LPDCB lpdcb = &(lpCC->dcb);
      char  temp[40];
@@ -3208,11 +3212,12 @@ BOOL WINAPI GetDefaultCommConfigA(
  *   if the device is not found or the buffer is too small.
  */
 BOOL WINAPI GetDefaultCommConfigW(
-	LPCWSTR lpszName, /* The unicode name of the device targeted for configuration. */
-	LPCOMMCONFIG lpCC, /* The default configuration for the device. */
-    LPDWORD lpdwSize /* Initially the size of the default configuration buffer,
-					    afterwards the number of bytes copied to the buffer or the needed size of the buffer. */
-) {
+    LPCWSTR      lpszName, /* [in] The unicode name of the device targeted for configuration. */
+    LPCOMMCONFIG lpCC,     /* [out] The default configuration for the device. */
+    LPDWORD      lpdwSize) /* [in/out] Initially the size of the default configuration buffer,
+			      afterwards the number of bytes copied to the buffer or
+                              the needed size of the buffer. */
+{
 	BOOL ret = FALSE;
 	LPSTR	lpszNameA;
 
diff --git a/dlls/kernel/debugger.c b/dlls/kernel/debugger.c
index 0db2981bedb..cfcb1c47852 100644
--- a/dlls/kernel/debugger.c
+++ b/dlls/kernel/debugger.c
@@ -27,8 +27,8 @@ DEFAULT_DEBUG_CHANNEL(debugstr);
  *  Returns true if a debug event occurred and false if the call timed out.
  */
 BOOL WINAPI WaitForDebugEvent(
-	 LPDEBUG_EVENT event, /* Address of structure for event information. */
-	 DWORD timeout /* Number of milliseconds to wait for event. */)
+    LPDEBUG_EVENT event,   /* [out] Address of structure for event information. */
+    DWORD         timeout) /* [in] Number of milliseconds to wait for event. */
 {
     BOOL ret;
     DWORD res;
@@ -132,9 +132,9 @@ BOOL WINAPI WaitForDebugEvent(
  *  and thread are valid.
  */
 BOOL WINAPI ContinueDebugEvent(
-	DWORD pid, /* The id of the process to continue. */
-	DWORD tid, /* The id of the thread to continue. */
-	DWORD status /* The rule to apply to unhandled exeptions. */)
+    DWORD pid,    /* [in] The id of the process to continue. */
+    DWORD tid,    /* [in] The id of the thread to continue. */
+    DWORD status) /* [in] The rule to apply to unhandled exeptions. */
 {
     BOOL ret;
     SERVER_START_REQ
@@ -160,7 +160,7 @@ BOOL WINAPI ContinueDebugEvent(
  *  True if the debugger was attached to process.
  */
 BOOL WINAPI DebugActiveProcess(
-	DWORD pid /* The process to be debugged. */)
+    DWORD pid) /* [in] The process to be debugged. */
 {
     BOOL ret;
     SERVER_START_REQ
@@ -182,7 +182,7 @@ BOOL WINAPI DebugActiveProcess(
  *  and program log.
  */
 void WINAPI OutputDebugStringA(
-        LPCSTR str /* The message to be logged and given to the debugger. */)
+    LPCSTR str) /* [in] The message to be logged and given to the debugger. */
 {
     SERVER_START_REQ
     {
@@ -204,7 +204,7 @@ void WINAPI OutputDebugStringA(
  *  and program log.
  */
 void WINAPI OutputDebugStringW(
-	LPCWSTR str /* The message to be logged and given to the debugger. */)
+    LPCWSTR str) /* [in] The message to be logged and given to the debugger. */
 {
     SERVER_START_REQ
     {
@@ -226,7 +226,7 @@ void WINAPI OutputDebugStringW(
  *  and program log.
  */
 void WINAPI OutputDebugString16(
-        LPCSTR str /* The message to be logged and given to the debugger.*/)
+    LPCSTR str) /* [in] The message to be logged and given to the debugger. */
 {
     OutputDebugStringA( str );
 }
@@ -255,7 +255,7 @@ void WINAPI DebugBreak(void)
  *  Only 386 compatible processors implemented.
  */
 void WINAPI DebugBreak16(
-        CONTEXT86 *context /* A pointer to the 386 compatible processor state. */)
+    CONTEXT86 *context) /* [in/out] A pointer to the 386 compatible processor state. */
 {
 #ifdef __i386__
     EXCEPTION_RECORD rec;
diff --git a/dlls/kernel/time.c b/dlls/kernel/time.c
index 97cb6e672e4..28df8bdc601 100644
--- a/dlls/kernel/time.c
+++ b/dlls/kernel/time.c
@@ -41,7 +41,8 @@ static int TIME_GetBias( time_t utc, int *pdaylight)
  *  True if the time was set, false if the time was invalid or the
  *  necessary permissions were not held.
  */
-BOOL WINAPI SetLocalTime( const SYSTEMTIME *systime /* The desired local time. */ )
+BOOL WINAPI SetLocalTime(
+    const SYSTEMTIME *systime) /* [in] The desired local time. */
 {
     struct timeval tv;
     struct tm t;
@@ -98,9 +99,9 @@ BOOL WINAPI SetLocalTime( const SYSTEMTIME *systime /* The desired local time. *
  *  (also the signature is wrong it should have a return type of BOOL)
  */
 DWORD WINAPI GetSystemTimeAdjustment(
-	 LPDWORD lpTimeAdjustment, /* The clock adjustment per interupt in 100's of nanoseconds. */
-	 LPDWORD lpTimeIncrement, /* The time between clock interupts in 100's of nanoseconds. */
-	 LPBOOL lpTimeAdjustmentDisabled /* The clock synchonisation has been disabled. */ )
+    LPDWORD lpTimeAdjustment,         /* [out] The clock adjustment per interupt in 100's of nanoseconds. */
+    LPDWORD lpTimeIncrement,          /* [out] The time between clock interupts in 100's of nanoseconds. */
+    LPBOOL  lpTimeAdjustmentDisabled) /* [out] The clock synchonisation has been disabled. */
 {
     *lpTimeAdjustment = 0;
     *lpTimeIncrement = 0;
@@ -120,7 +121,7 @@ DWORD WINAPI GetSystemTimeAdjustment(
  *  necessary permissions were not held.
  */
 BOOL WINAPI SetSystemTime(
-    const SYSTEMTIME *systime /* The desired system time. */)
+    const SYSTEMTIME *systime) /* [in] The desired system time. */
 {
     struct timeval tv;
     struct timezone tz;
@@ -181,7 +182,7 @@ BOOL WINAPI SetSystemTime(
  *  The daylight savings time standard or TIME_ZONE_ID_INVALID if the call failed.
  */
 DWORD WINAPI GetTimeZoneInformation(
-	LPTIME_ZONE_INFORMATION tzinfo /* The time zone structure to be filled in. */)
+    LPTIME_ZONE_INFORMATION tzinfo) /* [out] The time zone structure to be filled in. */
 {
     time_t gmt;
     int bias, daylight;
@@ -213,7 +214,7 @@ DWORD WINAPI GetTimeZoneInformation(
  *  Use the obsolete unix timezone structure and tz_dsttime member.
  */
 BOOL WINAPI SetTimeZoneInformation(
-	const LPTIME_ZONE_INFORMATION tzinfo /* The new time zone. */)
+    const LPTIME_ZONE_INFORMATION tzinfo) /* [in] The new time zone. */
 {
     struct timezone tz;
 
@@ -241,9 +242,9 @@ BOOL WINAPI SetTimeZoneInformation(
  *  Does not handle daylight savings time adjustments correctly.
  */
 BOOL WINAPI SystemTimeToTzSpecificLocalTime(
-	LPTIME_ZONE_INFORMATION lpTimeZoneInformation, /* The desired time zone. */
-	LPSYSTEMTIME lpUniversalTime, /* The utc time to base local time on. */
-	LPSYSTEMTIME lpLocalTime /* The local time in the time zone. */)
+    LPTIME_ZONE_INFORMATION lpTimeZoneInformation, /* [in] The desired time zone. */
+    LPSYSTEMTIME            lpUniversalTime,       /* [in] The utc time to base local time on. */
+    LPSYSTEMTIME            lpLocalTime)           /* [out] The local time in the time zone. */
 {
   FIXME(":stub\n");
   SetLastError(ERROR_CALL_NOT_IMPLEMENTED);
@@ -257,7 +258,7 @@ BOOL WINAPI SystemTimeToTzSpecificLocalTime(
  *  Fills in a file time structure with the current time in UTC format.
  */
 VOID WINAPI GetSystemTimeAsFileTime(
-	LPFILETIME time /* The file time struct to be filled with the system time. */)
+    LPFILETIME time) /* [out] The file time struct to be filled with the system time. */
 {
     NtQuerySystemTime( (LARGE_INTEGER *)time );
 }
@@ -298,11 +299,11 @@ static void TIME_ClockTimeToFileTime(clock_t unix_time, LPFILETIME filetime)
  *  lpCreationTime, lpExitTime are NOT INITIALIZED.
  */
 BOOL WINAPI GetProcessTimes(
-	HANDLE hprocess, /* The process to be queried (obtained from PROCESS_QUERY_INFORMATION). */
-	LPFILETIME lpCreationTime, /* The creation time of the process. */
-	LPFILETIME lpExitTime, /* The exit time of the process if exited. */
-	LPFILETIME lpKernelTime, /* The time spent in kernal routines in 100's of nanoseconds. */
-	LPFILETIME lpUserTime /* The time spent in user routines in 100's of nanoseconds. */)
+    HANDLE     hprocess,       /* [in] The process to be queried (obtained from PROCESS_QUERY_INFORMATION). */
+    LPFILETIME lpCreationTime, /* [out] The creation time of the process. */
+    LPFILETIME lpExitTime,     /* [out] The exit time of the process if exited. */
+    LPFILETIME lpKernelTime,   /* [out] The time spent in kernal routines in 100's of nanoseconds. */
+    LPFILETIME lpUserTime)     /* [out] The time spent in user routines in 100's of nanoseconds. */
 {
     struct tms tms;
 
diff --git a/dlls/oleaut32/dispatch.c b/dlls/oleaut32/dispatch.c
index 0192046a1aa..86d08fc46f0 100644
--- a/dlls/oleaut32/dispatch.c
+++ b/dlls/oleaut32/dispatch.c
@@ -25,7 +25,7 @@ DECLARE_DEBUG_CHANNEL(typelib);
 
 
 /******************************************************************************
- *         DispInvoke    (OLEAUT32.30)
+ *		DispInvoke (OLEAUT32.30)
  *
  *
  * Calls method of an object through its IDispatch interface.
@@ -37,16 +37,15 @@ DECLARE_DEBUG_CHANNEL(typelib);
  *
  * 		S_OK on success.
  */
-HRESULT WINAPI
-DispInvoke(VOID*            _this,          /* object instance */
-           ITypeInfo*       ptinfo,         /* object's type info */
-           DISPID           dispidMember,   /* member id */
-           USHORT           wFlags,         /* kind of method call */
-           DISPPARAMS*      pparams,        /* array of arguments */
-           VARIANT*         pvarResult,     /* result of method call */
-           EXCEPINFO*       pexcepinfo,     /* information about exception */
-           UINT*            puArgErr        /* index of bad argument(if any) */
-          )
+HRESULT WINAPI DispInvoke(
+	VOID       *_this,        /* [in] object instance */
+	ITypeInfo  *ptinfo,       /* [in] object's type info */
+	DISPID      dispidMember, /* [in] member id */
+	USHORT      wFlags,       /* [in] kind of method call */
+	DISPPARAMS *pparams,      /* [in] array of arguments */
+	VARIANT    *pvarResult,   /* [out] result of method call */
+	EXCEPINFO  *pexcepinfo,   /* [out] information about exception */
+	UINT       *puArgErr)     /* [out] index of bad argument(if any) */
 {
     HRESULT hr = E_FAIL;
 
@@ -68,7 +67,7 @@ DispInvoke(VOID*            _this,          /* object instance */
 
 
 /******************************************************************************
- *         DispGetIDsOfNames (OLEAUT32.29)
+ *		DispGetIDsOfNames (OLEAUT32.29)
  *
  * Convert a set of names to dispids, based on information 
  * contained in object's type library.
@@ -80,11 +79,11 @@ DispInvoke(VOID*            _this,          /* object instance */
  *
  * 		S_OK on success.
  */
-HRESULT WINAPI
-DispGetIDsOfNames(ITypeInfo* ptinfo,
-                  OLECHAR**  rgszNames,
-                  UINT       cNames,
-                  DISPID*    rgdispid)
+HRESULT WINAPI DispGetIDsOfNames(
+	ITypeInfo  *ptinfo,    /* [in] */
+	OLECHAR   **rgszNames, /* [in] */
+	UINT        cNames,    /* [in] */
+	DISPID     *rgdispid)  /* [out] */
 {
     HRESULT hr = E_FAIL;
 
@@ -97,7 +96,7 @@ DispGetIDsOfNames(ITypeInfo* ptinfo,
 }
 
 /******************************************************************************
- *         DispGetParam    (OLEAUT32.30)
+ *		DispGetParam (OLEAUT32.28)
  *
  * Retrive a parameter from a DISPPARAMS structures and coerce it to
  * specified variant type
@@ -109,10 +108,12 @@ DispGetIDsOfNames(ITypeInfo* ptinfo,
  *
  * 		S_OK on success.
  */
-HRESULT WINAPI
-DispGetParam(DISPPARAMS*    pdispparams,
-             UINT           position,
-             VARTYPE        vtTarg)
+HRESULT WINAPI DispGetParam(
+	DISPPARAMS *pdispparams, /* [in] */
+	UINT        position,    /* [in] */
+	VARTYPE     vtTarg,      /* [in] */
+	VARIANT    *pvarResult,  /* [out] */
+	UINT       *puArgErr)    /* [out] */
 {
     HRESULT hr = E_FAIL;
 
diff --git a/files/file.c b/files/file.c
index f5ed7c4acc4..734e7f4c6c3 100644
--- a/files/file.c
+++ b/files/file.c
@@ -339,13 +339,13 @@ HANDLE FILE_CreateDevice( int client_id, DWORD access, LPSECURITY_ATTRIBUTES sa
  *
  * PARAMS
  *
- * filename     [I] pointer to filename to be accessed
- * access       [I] access mode requested
- * sharing      [I] share mode
- * sa           [I] pointer to security attributes
- * creation     [I] how to create the file
- * attributes   [I] attributes for newly created file
- * template     [I] handle to file with extended attributes to copy
+ * filename     [in] pointer to filename to be accessed
+ * access       [in] access mode requested
+ * sharing      [in] share mode
+ * sa           [in] pointer to security attributes
+ * creation     [in] how to create the file
+ * attributes   [in] attributes for newly created file
+ * template     [in] handle to file with extended attributes to copy
  *
  * RETURNS
  *   Success: Open handle to specified file
@@ -1141,10 +1141,10 @@ HFILE WINAPI _lclose( HFILE hFile )
  *
  */
 BOOL WINAPI GetOverlappedResult(
-    HANDLE hFile,              /* [I] handle of file to check on */
-    LPOVERLAPPED lpOverlapped, /* [I/O] pointer to overlapped  */
-    LPDWORD lpTransferred,     /* [I/O] number of bytes transfered  */
-    BOOL bWait                 /* [I] wait for the transfer to complete ? */
+    HANDLE hFile,              /* [in] handle of file to check on */
+    LPOVERLAPPED lpOverlapped, /* [in/out] pointer to overlapped  */
+    LPDWORD lpTransferred,     /* [in/out] number of bytes transfered  */
+    BOOL bWait                 /* [in] wait for the transfer to complete ? */
 ) {
     DWORD r;
 
-- 
GitLab