* TraceImpl.cs:

	  - Remove the "lock-free" code, which (come to think of it) would be 
	    faulty in a multithreaded environment anyway because the "other"
	    threads won't wait (block) until initialization is complete...
	  - Add InitOnce() calls to all public properties, so that
	    `Debug.AutoFlush=true' will cause InitOnce() to be loaded, lest
	    a `Debug.AutoFlush=true; Debug.Listeners.Add(...)` sequence cause
	    .AutoFlush=true to be ignored as the InitOnce implied by .Add() 
	    will "overwrite" the .AutoFlush.
	  - Split up TraceListenerCollection creation from DefaultTraceListener 
	    addition, as TraceListenerCollection.Add() calls back into 
	    TraceImpl, so if these aren't separate we get an infinite loop
	    and never actually initialize anything.
	  - Fixes #424370.
	* TraceListenerCollection.cs: Add a TraceListenerCollection(bool)
	  constructor, which provides a way to NOT add the
	  DefaultTraceListener (needed by TraceImpl; see above).

svn path=/trunk/mcs/; revision=112596

1 files changed, 20 insertions, 0 deletions

diff --git a/mcs/class/System/System.Diagnostics/ChangeLog b/mcs/class/System/System.Diagnostics/ChangeLog
index 152369bec4e..e42d4d68583 100644
--- a/mcs/class/System/System.Diagnostics/ChangeLog
+++ b/mcs/class/System/System.Diagnostics/ChangeLog
@@ -1,3 +1,23 @@
+2008-09-09  Jonathan Pryor  <jpryor@novell.com>
+
+	* TraceImpl.cs:
+	  - Remove the "lock-free" code, which (come to think of it) would be 
+	    faulty in a multithreaded environment anyway because the "other"
+	    threads won't wait (block) until initialization is complete...
+	  - Add InitOnce() calls to all public properties, so that
+	    `Debug.AutoFlush=true' will cause InitOnce() to be loaded, lest
+	    a `Debug.AutoFlush=true; Debug.Listeners.Add(...)` sequence cause
+	    .AutoFlush=true to be ignored as the InitOnce implied by .Add() 
+	    will "overwrite" the .AutoFlush.
+	  - Split up TraceListenerCollection creation from DefaultTraceListener 
+	    addition, as TraceListenerCollection.Add() calls back into 
+	    TraceImpl, so if these aren't separate we get an infinite loop
+	    and never actually initialize anything.
+	  - Fixes #424370.
+	* TraceListenerCollection.cs: Add a TraceListenerCollection(bool)
+	  constructor, which provides a way to NOT add the
+	  DefaultTraceListener (needed by TraceImpl; see above).
+
 2008-08-06  Sebastien Pouliot  <sebastien@ximian.com>
 
 	* Debug_2_1.cs: Add missing [Conditional] attributes.