Re: cvs commit: logging-log4net/src/Util PatternString.cs

2005-05-12 Thread Stefan Bodewig
On Wed, 11 May 2005, Nicko Cadell [EMAIL PROTECTED] wrote:

 I use cygwin cvs which is configured to work in UNIX mode on my
 WinXP box. This is not really by design, its just the way it is
 right now.

I don't want to make an issue of this, but I'm curious.  Why are you
using the UNIX mode instead of DOS?  Is there some other tool that
works better that way?  Is it just for historic reasons?

Stefan


Re: cvs commit: logging-log4net/src/Util PatternString.cs

2005-05-12 Thread Stefan Bodewig
On Thu, 12 May 2005, Nicko Cadell [EMAIL PROTECTED] wrote:

 I haven't played around with subversion too much and I don't know
 how it deals with the LF / CRLF issue, I guess we will find out ;)

Unlike CVS, Subversion treats all files as binary files - and doesn't
do any keyword expansion either - unless you tell it to.  You have to
set properties on the individual files to make it translate line-feeds
or make any other changes.

For people working on Unix it is a bit annoying if you have cariage
returns in the files and as soon as you have a committer working on
Unix you may end up with files that have mixed line-ends since the
code she/he added would have line-feeds only.

Where it becomes a real problem is when you have Unix shell scripts in
the module since the shell will see the cariage return in the she-bang
line and fail with something like file not found /bin/sh^M where ^M
is a \r and easy to overlook.

Stefan


cvs commit: logging-log4net/src/Util PatternString.cs

2005-05-11 Thread nicko
nicko   2005/05/11 06:21:05

  Modified:src/Layout PatternLayout.cs
   src/Util PatternString.cs
  Log:
  Fixed preallocated hashtable size for static patterns
  
  Revision  ChangesPath
  1.21  +1 -1  logging-log4net/src/Layout/PatternLayout.cs
  
  Index: PatternLayout.cs
  ===
  RCS file: /home/cvs/logging-log4net/src/Layout/PatternLayout.cs,v
  retrieving revision 1.20
  retrieving revision 1.21
  diff -u -r1.20 -r1.21
  --- PatternLayout.cs  31 Jan 2005 02:32:10 -  1.20
  +++ PatternLayout.cs  11 May 2005 13:21:05 -  1.21
  @@ -766,7 +766,7 @@
/// /remarks

static PatternLayout()

{

  - s_globalRulesRegistry = new Hashtable(35);

  + s_globalRulesRegistry = new Hashtable(45);

   

s_globalRulesRegistry.Add(literal, 
typeof(log4net.Util.PatternStringConverters.LiteralPatternConverter));

s_globalRulesRegistry.Add(newline, 
typeof(log4net.Util.PatternStringConverters.NewLinePatternConverter));

  
  
  
  1.13  +2 -2  logging-log4net/src/Util/PatternString.cs
  
  Index: PatternString.cs
  ===
  RCS file: /home/cvs/logging-log4net/src/Util/PatternString.cs,v
  retrieving revision 1.12
  retrieving revision 1.13
  diff -u -r1.12 -r1.13
  --- PatternString.cs  31 Jan 2005 22:56:49 -  1.12
  +++ PatternString.cs  11 May 2005 13:21:05 -  1.13
  @@ -39,7 +39,7 @@
/// This class functions similarly to the see 
cref=log4net.Layout.PatternLayout/

/// in that it accepts a pattern and renders it to a string. Unlike the 

/// see cref=log4net.Layout.PatternLayout/ however the 
cPatternString/c

  - /// does does not render properties of a specific see 
cref=LoggingEvent/ but

  + /// does not render the properties of a specific see 
cref=LoggingEvent/ but

/// of the process in general.

/// /para

/// para

  @@ -281,7 +281,7 @@
/// /summary

static PatternString()

{

  - s_globalRulesRegistry = new Hashtable(10);

  + s_globalRulesRegistry = new Hashtable(15);

   

s_globalRulesRegistry.Add(appdomain, 
typeof(AppDomainPatternConverter));

s_globalRulesRegistry.Add(date, 
typeof(DatePatternConverter));

  
  
  


cvs commit: logging-log4net/src/Util PatternString.cs

2004-05-30 Thread nicko
nicko   2004/05/30 03:48:26

  Modified:src/Util/PatternStringConverters DatePatternConverter.cs
NewLinePatternConverter.cs
   src/Util PatternString.cs
  Log:
  Updated doc comments
  
  Revision  ChangesPath
  1.4   +16 -0 
logging-log4net/src/Util/PatternStringConverters/DatePatternConverter.cs
  
  Index: DatePatternConverter.cs
  ===
  RCS file: 
/home/cvs/logging-log4net/src/Util/PatternStringConverters/DatePatternConverter.cs,v
  retrieving revision 1.3
  retrieving revision 1.4
  diff -u -r1.3 -r1.4
  --- DatePatternConverter.cs   23 Feb 2004 03:18:05 -  1.3
  +++ DatePatternConverter.cs   30 May 2004 10:48:26 -  1.4
  @@ -36,6 +36,22 @@

#region Implementation of IOptionHandler
   
  + /// summary
  + /// Initialize the converter options
  + /// /summary
  + /// remarks
  + /// para
  + /// This is part of the see cref=IOptionHandler/ delayed 
object
  + /// activation scheme. The see cref=ActivateOptions/ method 
must 
  + /// be called on this object after the configuration properties 
have
  + /// been set. Until see cref=ActivateOptions/ is called this
  + /// object is in an undefined state and must not be used. 
  + /// /para
  + /// para
  + /// If any of the configuration properties are modified then 
  + /// see cref=ActivateOptions/ must be called again.
  + /// /para
  + /// /remarks
public void ActivateOptions()
{
string dateFormatStr = Option;
  
  
  
  1.3   +16 -0 
logging-log4net/src/Util/PatternStringConverters/NewLinePatternConverter.cs
  
  Index: NewLinePatternConverter.cs
  ===
  RCS file: 
/home/cvs/logging-log4net/src/Util/PatternStringConverters/NewLinePatternConverter.cs,v
  retrieving revision 1.2
  retrieving revision 1.3
  diff -u -r1.2 -r1.3
  --- NewLinePatternConverter.cs16 Feb 2004 02:10:55 -  1.2
  +++ NewLinePatternConverter.cs30 May 2004 10:48:26 -  1.3
  @@ -33,6 +33,22 @@
{
#region Implementation of IOptionHandler
   
  + /// summary
  + /// Initialize the converter options
  + /// /summary
  + /// remarks
  + /// para
  + /// This is part of the see cref=IOptionHandler/ delayed 
object
  + /// activation scheme. The see cref=ActivateOptions/ method 
must 
  + /// be called on this object after the configuration properties 
have
  + /// been set. Until see cref=ActivateOptions/ is called this
  + /// object is in an undefined state and must not be used. 
  + /// /para
  + /// para
  + /// If any of the configuration properties are modified then 
  + /// see cref=ActivateOptions/ must be called again.
  + /// /para
  + /// /remarks
public void ActivateOptions()
{
if (string.Compare(Option, DOS, true, 
System.Globalization.CultureInfo.InvariantCulture) == 0)
  
  
  
  1.4   +14 -1 logging-log4net/src/Util/PatternString.cs
  
  Index: PatternString.cs
  ===
  RCS file: /home/cvs/logging-log4net/src/Util/PatternString.cs,v
  retrieving revision 1.3
  retrieving revision 1.4
  diff -u -r1.3 -r1.4
  --- PatternString.cs  23 Feb 2004 03:18:04 -  1.3
  +++ PatternString.cs  30 May 2004 10:48:26 -  1.4
  @@ -147,8 +147,21 @@
#region Implementation of IOptionHandler
   
/// summary
  - /// Does not do anything as options become effective 
immediately.
  + /// Initialize component options
/// /summary
  + /// remarks
  + /// para
  + /// This is part of the see cref=IOptionHandler/ delayed 
object
  + /// activation scheme. The see cref=ActivateOptions/ method 
must 
  + /// be called on this object after the configuration properties 
have
  + /// been set. Until see cref=ActivateOptions/ is called this
  + /// object is in an undefined state and must not be used. 
  + /// /para
  + /// para
  + /// If any of the configuration properties are modified then 
  + /// see cref=ActivateOptions/ must be called again.
  + /// /para
  + /// /remarks
virtual public void ActivateOptions() 
{
// nothing to do.