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/06/27 19:53:28 UTC

cvs commit: logging-log4net/src/Repository/Hierarchy XmlHierarchyConfigurator.cs

nicko       2004/06/27 10:53:28

  Modified:    src/Repository/Hierarchy XmlHierarchyConfigurator.cs
  Log:
  Fixed NullReferenceException thrown when parsing an unknown or inherited level.
  The exception was thrown from an incorrectly placed internal log message.
  
  Revision  Changes    Path
  1.5       +6 -2      logging-log4net/src/Repository/Hierarchy/XmlHierarchyConfigurator.cs
  
  Index: XmlHierarchyConfigurator.cs
  ===================================================================
  RCS file: /home/cvs/logging-log4net/src/Repository/Hierarchy/XmlHierarchyConfigurator.cs,v
  retrieving revision 1.4
  retrieving revision 1.5
  diff -u -r1.4 -r1.5
  --- XmlHierarchyConfigurator.cs	30 May 2004 10:52:40 -0000	1.4
  +++ XmlHierarchyConfigurator.cs	27 Jun 2004 17:53:28 -0000	1.5
  @@ -453,6 +453,7 @@
   				} 
   				else 
   				{
  +					LogLog.Debug("XmlConfigurator: Logger [" + loggerName + "] level set to inherit from parent.");	
   					log.Level = null;
   				}
   			} 
  @@ -461,10 +462,13 @@
   				log.Level = log.Hierarchy.LevelMap[levelStr];
   				if (log.Level == null)
   				{
  -					LogLog.Error("XmlConfigurator: Undefined level [" + levelStr + "] on Logger [" + log.Name + "].");
  +					LogLog.Error("XmlConfigurator: Undefined level [" + levelStr + "] on Logger [" + loggerName + "].");
  +				}
  +				else
  +				{
  +					LogLog.Debug("XmlConfigurator: Logger [" + loggerName + "] level set to [name=\"" + log.Level.Name + "\",value=" + log.Level.Value + "].");	
   				}
   			}
  -			LogLog.Debug("XmlConfigurator: Logger [" + loggerName + "] level set to [name=\"" + log.Level.Name + "\",value=" + log.Level.Value + "].");	
   		}
   
   		/// <summary>