Author: tkreuzer
Date: Wed Jun 15 13:33:48 2011
New Revision: 52246

URL: http://svn.reactos.org/svn/reactos?rev=52246&view=rev
Log:
[RTL]
Fix useage of volatile in casts.

Modified:
    trunk/reactos/lib/rtl/workitem.c

Modified: trunk/reactos/lib/rtl/workitem.c
URL: 
http://svn.reactos.org/svn/reactos/trunk/reactos/lib/rtl/workitem.c?rev=52246&r1=52245&r2=52246&view=diff
==============================================================================
--- trunk/reactos/lib/rtl/workitem.c [iso-8859-1] (original)
+++ trunk/reactos/lib/rtl/workitem.c [iso-8859-1] Wed Jun 15 13:33:48 2011
@@ -45,7 +45,7 @@
 static LONG ThreadPoolIOWorkerThreadsRequests;
 static LONG ThreadPoolIOWorkerThreadsLongRequests;
 
-#define IsThreadPoolInitialized() ((volatile LONG)ThreadPoolInitialized == 1)
+#define IsThreadPoolInitialized() (*((volatile LONG*)&ThreadPoolInitialized) 
== 1)
 
 static NTSTATUS
 RtlpInitializeThreadPool(VOID)
@@ -614,8 +614,8 @@
             /* FIXME - figure out an effective method to determine if it's 
appropriate to
                        lower the number of threads. For now let's always 
terminate if there's
                        at least one thread and no queued items. */
-            Terminate = ((volatile LONG)ThreadPoolIOWorkerThreads - (volatile 
LONG)ThreadPoolIOWorkerThreadsLongRequests >= WORKERTHREAD_CREATION_THRESHOLD) 
&&
-                        ((volatile LONG)ThreadPoolIOWorkerThreadsRequests == 
0);
+            Terminate = (*((volatile LONG*)&ThreadPoolIOWorkerThreads) - 
*((volatile LONG*)&ThreadPoolIOWorkerThreadsLongRequests) >= 
WORKERTHREAD_CREATION_THRESHOLD) &&
+                        (*((volatile LONG*)&ThreadPoolIOWorkerThreadsRequests) 
== 0);
 
             if (Terminate)
             {
@@ -718,7 +718,7 @@
             {
                 /* FIXME - we might want to optimize this */
                 if (TimeoutCount++ > 2 &&
-                    (volatile LONG)ThreadPoolWorkerThreads - (volatile 
LONG)ThreadPoolWorkerThreadsLongRequests >= WORKERTHREAD_CREATION_THRESHOLD)
+                    *((volatile LONG*)&ThreadPoolWorkerThreads) - *((volatile 
LONG*)&ThreadPoolWorkerThreadsLongRequests) >= WORKERTHREAD_CREATION_THRESHOLD)
                 {
                     Terminate = TRUE;
                 }
@@ -819,7 +819,7 @@
                 /* Grow the thread pool */
                 Status = RtlpStartWorkerThread(RtlpIoWorkerThreadProc);
 
-                if (!NT_SUCCESS(Status) && (volatile 
LONG)ThreadPoolIOWorkerThreads != 0)
+                if (!NT_SUCCESS(Status) && *((volatile 
LONG*)&ThreadPoolIOWorkerThreads) != 0)
                 {
                     /* We failed to create the thread, but there's at least 
one there so
                        we can at least queue the request */
@@ -846,7 +846,7 @@
                 /* Grow the thread pool */
                 Status = RtlpStartWorkerThread(RtlpWorkerThreadProc);
 
-                if (!NT_SUCCESS(Status) && (volatile 
LONG)ThreadPoolWorkerThreads != 0)
+                if (!NT_SUCCESS(Status) && *((volatile 
LONG*)&ThreadPoolWorkerThreads) != 0)
                 {
                     /* We failed to create the thread, but there's at least 
one there so
                        we can at least queue the request */


Reply via email to