--- parser3/src/include/pa_threads.h	2001/01/29 11:53:42	1.2
+++ parser3/src/include/pa_threads.h	2001/03/19 15:29:38	1.7
@@ -1,5 +1,9 @@
-/*
-  $Id: pa_threads.h,v 1.2 2001/01/29 11:53:42 paf Exp $
+/** @file
+	Parser
+	Copyright (c) 2001 ArtLebedev Group (http://www.artlebedev.com)
+	Author: Alexander Petrosyan <paf@design.ru> (http://design.ru/paf)
+
+	$Id: pa_threads.h,v 1.7 2001/03/19 15:29:38 paf Exp $
 */
 
 #ifndef PA_THREADS_H
@@ -14,6 +18,7 @@
 
 #ifdef MULTITHREAD
 
+/// simple semaphore object
 class Mutex {
 	uint handle;
 public:
@@ -25,18 +30,41 @@ public:
 
 extern Mutex global_mutex;
 
+/** @brief
+	Helper to ensure paired Mutex::acquire() and Mutex::release().
+
+	Use it with SYNCHRONIZED macro
+*/
 class AutoSYNCHRONIZED {
+	bool thread_safe;
 public:
-	AutoSYNCHRONIZED() { global_mutex.acquire(); }
-	~AutoSYNCHRONIZED() { global_mutex.release(); }
-}
+	AutoSYNCHRONIZED(bool athread_safe) : thread_safe(athread_safe) { 
+		if(thread_safe)
+			global_mutex.acquire(); 
+	}
+	~AutoSYNCHRONIZED() { 
+		if(thread_safe)
+			global_mutex.release(); 
+	}
+};
+
+/** @brief
+	put it to first line of a function to ensure thread safety
 
-#define SYNCHRONIZED AutoSYNCHRONIZED autoSYNCHRONIZED()
+	@verbatim
+	void someclass::somefunc(...) { SYNCHRONIZED(thread_safe);
+		...
+	}
+	@endverbatim
 
+	considering \a thread_safe to be the object field to flag 
+	whether safety is really needed in this particular object instance
+*/
+#define SYNCHRONIZED(athread_safe) AutoSYNCHRONIZED autoSYNCHRONIZED(athread_safe)
 
 #else // not MULTITHREAD-ed
 
-#define SYNCHRONIZED /* do nothing */
+#define SYNCHRONIZED(athread_safe) /* do nothing */
 
 #endif