[ros-diffs] [hbelusca] 58922: [CSRSRV] - Code simplification. - Remove unneeded casts. - Fix comments.

Author: hbelusca
Date: Sat May  4 14:27:09 2013
New Revision: 58922

URL: http://svn.reactos.org/svn/reactos?rev=58922&view=rev
Log:
[CSRSRV]
- Code simplification.
- Remove unneeded casts.
- Fix comments.

Modified:
    trunk/reactos/subsystems/win32/csrsrv/api.c
    trunk/reactos/subsystems/win32/csrsrv/thredsup.c

Modified: trunk/reactos/subsystems/win32/csrsrv/api.c
URL:
http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/win32/csrsrv/ap…
==============================================================================
--- trunk/reactos/subsystems/win32/csrsrv/api.c	[iso-8859-1] (original)
+++ trunk/reactos/subsystems/win32/csrsrv/api.c	[iso-8859-1] Sat May  4 14:27:09 2013
@@ -63,7 +63,7 @@
     {
         /* We are beyond the Maximum Server ID */
         DPRINT1("CSRSS: %lx is invalid ServerDllIndex (%08x)\n", ServerId,
ServerDll);
-        ReplyMsg->Status = (ULONG)STATUS_ILLEGAL_FUNCTION;
+        ReplyMsg->Status = STATUS_ILLEGAL_FUNCTION;
         return STATUS_ILLEGAL_FUNCTION;
     }
     else
@@ -82,7 +82,7 @@
                     ((ServerDll->NameTable) &&
(ServerDll->NameTable[ApiId])) ?
                     ServerDll->NameTable[ApiId] : "*** UNKNOWN ***",
&ServerDll->Name);
             // DbgBreakPoint();
-            ReplyMsg->Status = (ULONG)STATUS_ILLEGAL_FUNCTION;
+            ReplyMsg->Status = STATUS_ILLEGAL_FUNCTION;
             return STATUS_ILLEGAL_FUNCTION;
         }
     }
@@ -155,7 +155,7 @@
         if (CsrProcess)
         {
             /* Reference the Process */
-            CsrLockedReferenceProcess(CsrThread->Process);
+            CsrLockedReferenceProcess(CsrProcess);
 
             /* Release the lock */
             CsrReleaseProcessLock();
@@ -432,7 +432,7 @@
             }
             else
             {
-                /* A bizare "success" code, just try again */
+                /* A strange "success" code, just try again */
                 DPRINT1("NtReplyWaitReceivePort returned \"success\"
status 0x%x\n", Status);
                 continue;
             }

Modified: trunk/reactos/subsystems/win32/csrsrv/thredsup.c
URL:
http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/win32/csrsrv/th…
==============================================================================
--- trunk/reactos/subsystems/win32/csrsrv/thredsup.c	[iso-8859-1] (original)
+++ trunk/reactos/subsystems/win32/csrsrv/thredsup.c	[iso-8859-1] Sat May  4 14:27:09
2013
@@ -958,10 +958,10 @@
     /* Start Loop */
     while (NextEntry != &CsrThreadHashTable[i])
     {
-        /* Get the Process */
+        /* Get the Thread */
         CurrentThread = CONTAINING_RECORD(NextEntry, CSR_THREAD, HashLinks);
 
-        /* Check for PID Match */
+        /* Check for TID Match */
         if ((CurrentThread->ClientId.UniqueThread == Tid) &&
             (CurrentThread->Flags & CsrThreadTerminated) == 0)
         {