Minor fixes

Author: vonhoff

Serilog.Sinks.RichTextBox.WinForms.Colored/Sinks/RichTextBoxForms/RichTextBoxSink.cs

@@ -79,8 +79,6 @@ public void Emit(LogEvent logEvent)
         {
             _buffer.Add(logEvent);
             Interlocked.Exchange(ref _hasNewMessages, 1);
-
-            // Signal the processing task that there are new messages to process
             _signal.Set();
         }
 
@@ -101,15 +99,16 @@ private void ProcessMessages(CancellationToken token)
                     var elapsed = now - lastFlush;
                     if (elapsed < flushInterval)
                     {
-                        // Wait for the remaining time, or until a new message arrives
                         var remaining = flushInterval - elapsed;
                         if (remaining > TimeSpan.Zero)
                         {
-                            _signal.WaitOne(remaining);
+                            Thread.Sleep(remaining);
                         }
                     }
 
-                    // Take a snapshot and flush
+                    Interlocked.Exchange(ref _hasNewMessages, 0);
+
+                    // Take a snapshot of the current buffer
                     _buffer.TakeSnapshot(snapshot);
                     builder.Clear();
                     foreach (var evt in snapshot)

Serilog.Sinks.RichTextBox.WinForms.Colored/Sinks/RichTextBoxForms/Rtf/RtfBuilder.cs

@@ -160,12 +160,6 @@ public void Clear()
         {
             _body.Clear();
 
-            // Prevent unbounded memory retention by the underlying StringBuilder.
-            // When very large log batches have been rendered, the StringBuilder can
-            // grow to hundreds of kilobytes or even megabytes. Although Clear()
-            // resets its length, the backing buffer (Capacity) is retained which
-            // keeps the memory alive until the next Gen-2 GC collection, causing
-            // memory bloat and noticeable GC pauses.
             const int maxRetainedBuilderSize = 64 * 1024;
             if (_body.Capacity > maxRetainedBuilderSize)
             {