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/api... ============================================================================== --- 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/thr... ============================================================================== --- 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) {