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/12/19 20:24:30 UTC

cvs commit: logging-log4net/src/Util/PatternStringConverters LiteralPatternConverter.cs

nicko       2004/12/19 11:24:30

  Modified:    src/Util/PatternStringConverters LiteralPatternConverter.cs
  Log:
  Updates suggested by FxCop.
  Minor naming changes to internal types.
  Perf improvements to type check + type cast code.
  
  Revision  Changes    Path
  1.4       +3 -2      logging-log4net/src/Util/PatternStringConverters/LiteralPatternConverter.cs
  
  Index: LiteralPatternConverter.cs
  ===================================================================
  RCS file: /home/cvs/logging-log4net/src/Util/PatternStringConverters/LiteralPatternConverter.cs,v
  retrieving revision 1.3
  retrieving revision 1.4
  diff -u -r1.3 -r1.4
  --- LiteralPatternConverter.cs	23 Feb 2004 03:18:05 -0000	1.3
  +++ LiteralPatternConverter.cs	19 Dec 2004 19:24:30 -0000	1.4
  @@ -35,10 +35,11 @@
   		/// </summary>
   		public override PatternConverter SetNext(PatternConverter pc)
   		{
  -			if (pc is LiteralPatternConverter)
  +			LiteralPatternConverter literalPc = pc as LiteralPatternConverter;
  +			if (literalPc != null)
   			{
   				// Combine the two adjacent literals together
  -				Option = Option + ((LiteralPatternConverter)pc).Option;
  +				Option = Option + literalPc.Option;
   
   				// We are the next converter now
   				return this;