You are viewing a plain text version of this content. The canonical link for it is here.
Posted to log4net-dev@logging.apache.org by ni...@apache.org on 2004/02/26 21:40:07 UTC
cvs commit: logging-log4net/src/Appender BufferingAppenderSkeleton.cs
nicko 2004/02/26 12:40:06
Modified: src/Appender BufferingAppenderSkeleton.cs
Log:
Changed to not create the cyclic buffer if the buffer size is less than 2
Revision Changes Path
1.4 +9 -2 logging-log4net/src/Appender/BufferingAppenderSkeleton.cs
Index: BufferingAppenderSkeleton.cs
===================================================================
RCS file: /home/cvs/logging-log4net/src/Appender/BufferingAppenderSkeleton.cs,v
retrieving revision 1.3
retrieving revision 1.4
diff -u -r1.3 -r1.4
--- BufferingAppenderSkeleton.cs 23 Feb 2004 03:18:02 -0000 1.3
+++ BufferingAppenderSkeleton.cs 26 Feb 2004 20:40:06 -0000 1.4
@@ -239,7 +239,14 @@
ErrorHandler.Error("Appender ["+Name+"] is Lossy but has no Evaluator. The buffer will never be sent!");
}
- m_cb = new CyclicBuffer(m_bufferSize);
+ if (m_bufferSize > 1)
+ {
+ m_cb = new CyclicBuffer(m_bufferSize);
+ }
+ else
+ {
+ m_cb = null;
+ }
}
#endregion Implementation of IOptionHandler
@@ -313,7 +320,7 @@
// sent immediately because there is not enough space in the buffer
// to buffer up more than 1 event. Therefore as a special case
// we don't use the buffer at all.
- if (m_bufferSize <= 1)
+ if (m_cb == null || m_bufferSize <= 1)
{
// Only send the event if we are in non lossy mode or the event is a triggering event
if ((!m_lossy) ||