Look and criticize, Aleric

indra/llcommon/llthread.cpp

@@ -102,7 +102,7 @@ void *APR_THREAD_FUNC LLThread::staticRun(apr_thread_t *apr_threadp, void *datap
 	// Setting mStatus to STOPPED is done non-thread-safe, so it's
 	// possible that the thread is deleted by another thread at
 	// the moment it happens... therefore make a copy here.
-	char const* name = threadp->mName;
+	volatile char const* name = threadp->mName.c_str();
 	
 	// We're done with the run function, this thread is done executing now.
 	threadp->mStatus = STOPPED;

indra/llcommon/llthread.h

@@ -128,7 +128,7 @@ protected:
 	LLCondition*		mRunCondition;
 
 	apr_thread_t		*mAPRThreadp;
-	EThreadStatus		mStatus;
+	volatile EThreadStatus		mStatus;
 	U32					mID;
 
 	friend void AIThreadLocalData::create(LLThread* threadp);

indra/newview/llviewerprecompiledheaders.h

@@ -96,7 +96,6 @@
 #include "llstack.h"
 #include "llstat.h"
 #include "llstl.h"
-#include "llstrider.h"
 #include "llstring.h"
 #include "llstringtable.h"
 #include "llsys.h"