You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ibatis.apache.org by rg...@apache.org on 2006/11/05 18:53:42 UTC

svn commit: r471478 - in /ibatis/trunk/cs/mapper/IBatisNet.Common/Logging: ConfigurationSectionHandler.cs ILog.cs Impl/ConsoleOutLogger.cs Impl/ConsoleOutLoggerFA.cs Impl/NoOpLogger.cs

Author: rgrabowski
Date: Sun Nov  5 09:53:41 2006
New Revision: 471478

URL: http://svn.apache.org/viewvc?view=rev&rev=471478
Log:
Added additional documentation information.

Modified:
    ibatis/trunk/cs/mapper/IBatisNet.Common/Logging/ConfigurationSectionHandler.cs
    ibatis/trunk/cs/mapper/IBatisNet.Common/Logging/ILog.cs
    ibatis/trunk/cs/mapper/IBatisNet.Common/Logging/Impl/ConsoleOutLogger.cs
    ibatis/trunk/cs/mapper/IBatisNet.Common/Logging/Impl/ConsoleOutLoggerFA.cs
    ibatis/trunk/cs/mapper/IBatisNet.Common/Logging/Impl/NoOpLogger.cs

Modified: ibatis/trunk/cs/mapper/IBatisNet.Common/Logging/ConfigurationSectionHandler.cs
URL: http://svn.apache.org/viewvc/ibatis/trunk/cs/mapper/IBatisNet.Common/Logging/ConfigurationSectionHandler.cs?view=diff&rev=471478&r1=471477&r2=471478
==============================================================================
--- ibatis/trunk/cs/mapper/IBatisNet.Common/Logging/ConfigurationSectionHandler.cs (original)
+++ ibatis/trunk/cs/mapper/IBatisNet.Common/Logging/ConfigurationSectionHandler.cs Sun Nov  5 09:53:41 2006
@@ -2,7 +2,7 @@
 #region Apache Notice
 /*****************************************************************************
  * $Header: $
- * $Revision: $
+ * $Revision$
  * $Date$
  * 
  * iBATIS.NET Data Mapper
@@ -34,8 +34,32 @@
 namespace IBatisNet.Common.Logging
 {
 	/// <summary>
-	/// Summary description for ConfigurationSectionHandler.
+	/// Used in an application's configuration file (App.Config or Web.Config) to configure the logging subsystem.
 	/// </summary>
+	/// <remarks>
+	/// <example>
+	/// An example configuration section that writes IBatisNet messages to the Console using the built-in Console Logger.
+	/// <code lang="XML" escaped="true">
+	/// <configuration>
+	///		<configSections>
+	///			<sectionGroup name="iBATIS">
+	///				<section name="logging" type="IBatisNet.Common.Logging.ConfigurationSectionHandler, IBatisNet.Common" />
+	///			</sectionGroup>	
+	///		</configSections>
+	///		<iBATIS>
+	///			<logging>
+	///				<logFactoryAdapter type="IBatisNet.Common.Logging.Impl.ConsoleOutLoggerFA, IBatisNet.Common">
+	///					<arg key="showLogName" value="true" />
+	///					<arg key="showDataTime" value="true" />
+	///					<arg key="level" value="ALL" />
+	///					<arg key="dateTimeFormat" value="yyyy/MM/dd HH:mm:ss:SSS" />
+	///				</logFactoryAdapter>
+	///			</logging>
+	///		</iBATIS>
+	/// </configuration>
+	/// </code> 
+	/// </example>
+	/// </remarks>
 	public class ConfigurationSectionHandler: IConfigurationSectionHandler
 	{
 
@@ -57,10 +81,15 @@
 		}
 
 		/// <summary>
-		/// 
+		/// Retrieves the <see cref="Type" /> of the logger the use by looking at the logFactoryAdapter element
+		/// of the logging configuration element.
 		/// </summary>
 		/// <param name="section"></param>
-		/// <returns></returns>
+		/// <returns>
+		/// A <see cref="LogSetting" /> object containing the specified type that implements 
+		/// <see cref="ILoggerFactoryAdapter" /> along with zero or more properties that will be 
+		/// passed to the logger factory adapter's constructor as an <see cref="IDictionary" />.
+		/// </returns>
 		private LogSetting ReadConfiguration( XmlNode section )
 		{
 			XmlNode logFactoryElement = section.SelectSingleNode( LOGFACTORYADAPTER_ELEMENT );
@@ -72,10 +101,10 @@
 			if ( factoryTypeString == string.Empty )
 				throw new ConfigurationException
 					( "Required Attribute '" 
-					+ LOGFACTORYADAPTER_ELEMENT_TYPE_ATTRIB 
-					+ "' not found in element '"
-					+ LOGFACTORYADAPTER_ELEMENT
-					+ "'"
+					  + LOGFACTORYADAPTER_ELEMENT_TYPE_ATTRIB 
+					  + "' not found in element '"
+					  + LOGFACTORYADAPTER_ELEMENT
+					  + "'"
 					);
 
 
@@ -88,7 +117,7 @@
 			{
 				throw new ConfigurationException
 					( "Unable to create type '" + factoryTypeString + "'"
-					, e
+					  , e
 					);
 			}
 			
@@ -98,7 +127,7 @@
 #if dotnet2
             properties = new NameValueCollection(StringComparer.InvariantCultureIgnoreCase);
 #else
-			properties = properties = new NameValueCollection( null, new CaseInsensitiveComparer() );
+			properties = new NameValueCollection( null, new CaseInsensitiveComparer() );
 #endif
 			foreach ( XmlNode propertyNode in propertyNodes )
 			{
@@ -112,10 +141,10 @@
 				{
 					throw new ConfigurationException
 						( "Required Attribute '" 
-						+ ARGUMENT_ELEMENT_KEY_ATTRIB 
-						+ "' not found in element '"
-						+ ARGUMENT_ELEMENT
-						+ "'"
+						  + ARGUMENT_ELEMENT_KEY_ATTRIB 
+						  + "' not found in element '"
+						  + ARGUMENT_ELEMENT
+						  + "'"
 						);
 				}
 				else
@@ -137,13 +166,16 @@
 		#region IConfigurationSectionHandler Members
 
 		/// <summary>
-		/// 
+		/// Verifies that the logFactoryAdapter element appears once in the configuration section.
 		/// </summary>
-		/// <param name="parent"></param>
-		/// <param name="configContext"></param>
-		/// <param name="section"></param>
-		/// <returns></returns>
-		public object Create(object parent, object configContext, System.Xml.XmlNode section)
+		/// <param name="parent">The parent of the current item.</param>
+		/// <param name="configContext">Additional information about the configuration process.</param>
+		/// <param name="section">The configuration section to apply an XPath query too.</param>
+		/// <returns>
+		/// A <see cref="LogSetting" /> object containing the specified logFactoryAdapter type
+		/// along with user supplied configuration properties.
+		/// </returns>
+		public object Create(object parent, object configContext, XmlNode section)
 		{
 			int logFactoryElementsCount = section.SelectNodes( LOGFACTORYADAPTER_ELEMENT ).Count;
 			

Modified: ibatis/trunk/cs/mapper/IBatisNet.Common/Logging/ILog.cs
URL: http://svn.apache.org/viewvc/ibatis/trunk/cs/mapper/IBatisNet.Common/Logging/ILog.cs?view=diff&rev=471478&r1=471477&r2=471478
==============================================================================
--- ibatis/trunk/cs/mapper/IBatisNet.Common/Logging/ILog.cs (original)
+++ ibatis/trunk/cs/mapper/IBatisNet.Common/Logging/ILog.cs Sun Nov  5 09:53:41 2006
@@ -2,7 +2,7 @@
 #region Apache Notice
 /*****************************************************************************
  * $Header: $
- * $Revision: $
+ * $Revision$
  * $Date$
  * 
  * iBATIS.NET Data Mapper
@@ -29,7 +29,7 @@
 namespace IBatisNet.Common.Logging
 {
 	/// <summary>
-	/// The 8 logging levels used by Log are (in order): 
+	/// The 7 logging levels used by Log are (in order): 
 	/// </summary>
 	public enum LogLevel
 	{
@@ -54,11 +54,11 @@
 		/// </summary>
 		Error = 4,
 		/// <summary>
-		/// 
+		
 		/// </summary>
 		Fatal = 5,
 		/// <summary>
-		/// 
+		/// Do not log anything.
 		/// </summary>
 		Off  = 6,
 	}

Modified: ibatis/trunk/cs/mapper/IBatisNet.Common/Logging/Impl/ConsoleOutLogger.cs
URL: http://svn.apache.org/viewvc/ibatis/trunk/cs/mapper/IBatisNet.Common/Logging/Impl/ConsoleOutLogger.cs?view=diff&rev=471478&r1=471477&r2=471478
==============================================================================
--- ibatis/trunk/cs/mapper/IBatisNet.Common/Logging/Impl/ConsoleOutLogger.cs (original)
+++ ibatis/trunk/cs/mapper/IBatisNet.Common/Logging/Impl/ConsoleOutLogger.cs Sun Nov  5 09:53:41 2006
@@ -2,7 +2,7 @@
 #region Apache Notice
 /*****************************************************************************
  * $Header: $
- * $Revision: $
+ * $Revision$
  * $Date$
  * 
  * iBATIS.NET Data Mapper
@@ -31,9 +31,9 @@
 namespace IBatisNet.Common.Logging.Impl
 {
 	/// <summary>
-	/// Logger sending everything to the standard output error stream.
+	/// Sends log messages to <see cref="Console.Out" />.
 	/// </summary>
-	public class ConsoleOutLogger: ILog
+	public class ConsoleOutLogger : ILog
 	{
 		private bool _showDateTime = false;
 		private bool _showLogName = false;
@@ -43,15 +43,15 @@
 		private bool _hasDateTimeFormat = false;
 
 		/// <summary>
-		/// 
+		/// Creates and initializes a logger that writes messages to <see cref="Console.Out" />.
 		/// </summary>
-		/// <param name="logName"></param>
-		/// <param name="logLevel"></param>
-		/// <param name="showDateTime">Include the current time in the log message </param>
-		/// <param name="showLogName">Include the instance name in the log message</param>
-		/// <param name="dateTimeFormat">The date and time format to use in the log message </param>
+		/// <param name="logName">The name, usually type name of the calling class, of the logger.</param>
+		/// <param name="logLevel">The current logging threshold. Messages recieved that are beneath this threshold will not be logged.</param>
+		/// <param name="showDateTime">Include the current time in the log message.</param>
+		/// <param name="showLogName">Include the instance name in the log message.</param>
+		/// <param name="dateTimeFormat">The date and time format to use in the log message.</param>
 		public ConsoleOutLogger( string logName, LogLevel logLevel
-			, bool showDateTime, bool showLogName, string dateTimeFormat)
+		                         , bool showDateTime, bool showLogName, string dateTimeFormat)
 		{
 			_logName = logName;
 			_currentLogLevel = logLevel;
@@ -66,15 +66,12 @@
 		}
 
 		/// <summary>
-		/// Do the actual logging.
-		/// This method assembles the message and write
-		/// the content of the message accumulated in the specified
-		/// StringBuffer to the appropriate output destination. The
-		/// default implementation writes to System.Console.Error.<p/>
+		/// Do the actual logging by constructing the log message using a <see cref="StringBuilder" /> then
+		/// sending the output to <see cref="Console.Out" />.
 		/// </summary>
-		/// <param name="level"></param>
-		/// <param name="message"></param>
-		/// <param name="e"></param>
+		/// <param name="level">The <see cref="LogLevel" /> of the message.</param>
+		/// <param name="message">The log message.</param>
+		/// <param name="e">An optional <see cref="Exception" /> associated with the message.</param>
 		private void Write( LogLevel level, object message, Exception e )
 		{
 			// Use a StringBuilder for better performance
@@ -108,15 +105,15 @@
 			// Append stack trace if not null
 			if ( e != null )
 			{
-				sb.AppendFormat( "\n{0}", e.ToString() );
+				sb.Append(Environment.NewLine).Append( e.ToString() );
 			}
 
 			// Print to the appropriate destination
-			System.Console.Out.WriteLine( sb.ToString() );			
+			Console.Out.WriteLine( sb.ToString() );
 		}
 
 		/// <summary>
-		/// Is the given log level currently enabled ?
+		/// Determines if the given log level is currently enabled.
 		/// </summary>
 		/// <param name="level"></param>
 		/// <returns></returns>
@@ -124,26 +121,30 @@
 		{
 			int iLevel = (int)level;
 			int iCurrentLogLevel = (int)_currentLogLevel;
-
+		
+			// return iLevel.CompareTo(iCurrentLogLevel); better ???
 			return ( iLevel >= iCurrentLogLevel );
 		}
 
 		#region ILog Members
 
 		/// <summary>
-		/// Log a debug message.
+		/// Log a <see cref="LogLevel.Debug" /> message.
 		/// </summary>
-		/// <param name="message"></param>
+		/// <param name="message">The message to log.</param>
 		public void Debug(object message)
 		{
 			Debug( message, null );
 		}
 
 		/// <summary>
-		/// 
+		/// Log a <see cref="LogLevel.Debug" /> message with an optional <see cref="Exception" />.
 		/// </summary>
-		/// <param name="message"></param>
-		/// <param name="e"></param>
+		/// <param name="message">The message to log.</param>
+		/// <param name="e">
+		/// The	<see cref="Exception" /> associated with the message. If there isn't any
+		/// <see cref="Exception" /> associated with the message, pass <see langword="null" />.
+		/// </param>
 		public void Debug(object message, Exception e)
 		{
 			if ( IsLevelEnabled( LogLevel.Debug ) )
@@ -153,19 +154,22 @@
 		}
 
 		/// <summary>
-		/// 
+		/// Log a <see cref="LogLevel.Error" /> message.
 		/// </summary>
-		/// <param name="message"></param>
+		/// <param name="message">The message to log.</param>
 		public void Error(object message)
 		{
 			Error( message, null );
 		}
 
 		/// <summary>
-		/// 
+		/// Log a <see cref="LogLevel.Error" /> message with an optional <see cref="Exception" />.
 		/// </summary>
-		/// <param name="message"></param>
-		/// <param name="e"></param>
+		/// <param name="message">The message to log.</param>
+		/// <param name="e">
+		/// The	<see cref="Exception" /> associated with the message. If there isn't any
+		/// <see cref="Exception" /> associated with the message, pass <see langword="null" />.
+		/// </param>
 		public void Error(object message, Exception e)
 		{
 			if ( IsLevelEnabled( LogLevel.Error ) )
@@ -175,19 +179,22 @@
 		}
 
 		/// <summary>
-		/// 
+		/// Log a <see cref="LogLevel.Fatal" /> message.
 		/// </summary>
-		/// <param name="message"></param>
+		/// <param name="message">The message to log.</param>
 		public void Fatal(object message)
 		{
 			Fatal( message, null );
 		}
 
 		/// <summary>
-		/// 
+		/// Log a <see cref="LogLevel.Fatal" /> message with an optional <see cref="Exception" />.
 		/// </summary>
-		/// <param name="message"></param>
-		/// <param name="e"></param>
+		/// <param name="message">The message to log.</param>
+		/// <param name="e">
+		/// The	<see cref="Exception" /> associated with the message. If there isn't any
+		/// <see cref="Exception" /> associated with the message, pass <see langword="null" />.
+		/// </param>
 		public void Fatal(object message, Exception e)
 		{
 			if ( IsLevelEnabled( LogLevel.Fatal ) )
@@ -197,19 +204,22 @@
 		}
 
 		/// <summary>
-		/// 
+		/// Log a <see cref="LogLevel.Info" /> message.
 		/// </summary>
-		/// <param name="message"></param>
+		/// <param name="message">The message to log.</param>
 		public void Info(object message)
 		{
 			Info( message, null );
 		}
 
 		/// <summary>
-		/// 
+		/// Log a <see cref="LogLevel.Info" /> message with an optional <see cref="Exception" />.
 		/// </summary>
-		/// <param name="message"></param>
-		/// <param name="e"></param>
+		/// <param name="message">The message to log.</param>
+		/// <param name="e">
+		/// The	<see cref="Exception" /> associated with the message. If there isn't any
+		/// <see cref="Exception" /> associated with the message, pass <see langword="null" />.
+		/// </param>
 		public void Info(object message, Exception e)
 		{
 			if ( IsLevelEnabled( LogLevel.Info ) )
@@ -219,19 +229,22 @@
 		}
 
 		/// <summary>
-		/// 
+		/// Log a <see cref="LogLevel.Warn" /> message.
 		/// </summary>
-		/// <param name="message"></param>
+		/// <param name="message">The message to log.</param>
 		public void Warn(object message)
 		{
 			Warn( message, null );
 		}
 
 		/// <summary>
-		/// 
+		/// Log a <see cref="LogLevel.Warn" /> message with an optional <see cref="Exception" />.
 		/// </summary>
-		/// <param name="message"></param>
-		/// <param name="e"></param>
+		/// <param name="message">The message to log.</param>
+		/// <param name="e">
+		/// The	<see cref="Exception" /> associated with the message. If there isn't any
+		/// <see cref="Exception" /> associated with the message, pass <see langword="null" />.
+		/// </param>
 		public void Warn(object message, Exception e)
 		{
 			if ( IsLevelEnabled( LogLevel.Warn ) )
@@ -241,7 +254,8 @@
 		}
 
 		/// <summary>
-		/// 
+		/// Returns <see langword="true" /> if the current <see cref="LogLevel" /> is greater than or
+		/// equal to <see cref="LogLevel.Debug" />. If it is, all messages will be sent to <see cref="Console.Out" />.
 		/// </summary>
 		public bool IsDebugEnabled
 		{
@@ -249,7 +263,9 @@
 		}
 
 		/// <summary>
-		/// 
+		/// Returns <see langword="true" /> if the current <see cref="LogLevel" /> is greater than or
+		/// equal to <see cref="LogLevel.Error" />. If it is, only messages with a <see cref="LogLevel" /> of
+		/// <see cref="LogLevel.Error" /> and <see cref="LogLevel.Fatal" /> will be sent to <see cref="Console.Out" />.
 		/// </summary>
 		public bool IsErrorEnabled
 		{
@@ -257,7 +273,9 @@
 		}
 
 		/// <summary>
-		/// 
+		/// Returns <see langword="true" /> if the current <see cref="LogLevel" /> is greater than or
+		/// equal to <see cref="LogLevel.Fatal" />. If it is, only messages with a <see cref="LogLevel" /> of
+		/// <see cref="LogLevel.Fatal" /> will be sent to <see cref="Console.Out" />.
 		/// </summary>
 		public bool IsFatalEnabled
 		{
@@ -265,7 +283,10 @@
 		}
 
 		/// <summary>
-		/// 
+		/// Returns <see langword="true" /> if the current <see cref="LogLevel" /> is greater than or
+		/// equal to <see cref="LogLevel.Info" />. If it is, only messages with a <see cref="LogLevel" /> of
+		/// <see cref="LogLevel.Info" />, <see cref="LogLevel.Warn" />, <see cref="LogLevel.Error" />, and 
+		/// <see cref="LogLevel.Fatal" /> will be sent to <see cref="Console.Out" />.
 		/// </summary>
 		public bool IsInfoEnabled
 		{
@@ -274,7 +295,10 @@
 
 
 		/// <summary>
-		/// 
+		/// Returns <see langword="true" /> if the current <see cref="LogLevel" /> is greater than or
+		/// equal to <see cref="LogLevel.Warn" />. If it is, only messages with a <see cref="LogLevel" /> of
+		/// <see cref="LogLevel.Warn" />, <see cref="LogLevel.Error" />, and <see cref="LogLevel.Fatal" /> 
+		/// will be sent to <see cref="Console.Out" />.
 		/// </summary>
 		public bool IsWarnEnabled
 		{

Modified: ibatis/trunk/cs/mapper/IBatisNet.Common/Logging/Impl/ConsoleOutLoggerFA.cs
URL: http://svn.apache.org/viewvc/ibatis/trunk/cs/mapper/IBatisNet.Common/Logging/Impl/ConsoleOutLoggerFA.cs?view=diff&rev=471478&r1=471477&r2=471478
==============================================================================
--- ibatis/trunk/cs/mapper/IBatisNet.Common/Logging/Impl/ConsoleOutLoggerFA.cs (original)
+++ ibatis/trunk/cs/mapper/IBatisNet.Common/Logging/Impl/ConsoleOutLoggerFA.cs Sun Nov  5 09:53:41 2006
@@ -2,7 +2,7 @@
 #region Apache Notice
 /*****************************************************************************
  * $Header: $
- * $Revision: $
+ * $Revision$
  * $Date$
  * 
  * iBATIS.NET Data Mapper
@@ -31,7 +31,7 @@
 namespace IBatisNet.Common.Logging.Impl
 {
 	/// <summary>
-	/// Description résumée de ConsoleOutLoggerFA.
+	/// Factory for creating <see cref="ILog" /> instances that write data to <see cref="Console.Out" />.
 	/// </summary>
 	public class ConsoleOutLoggerFA: ILoggerFactoryAdapter 
 	{
@@ -42,9 +42,10 @@
 		private string _dateTimeFormat = string.Empty;
 
 		/// <summary>
-		/// Constructor
+		/// Looks for level, showDateTime, showLogName, dateTimeFormat items from 
+		/// <paramref name="properties" /> for use when the GetLogger methods are called.
 		/// </summary>
-		/// <param name="properties"></param>
+		/// <param name="properties">Contains user supplied configuration information.</param>
 		public ConsoleOutLoggerFA(NameValueCollection properties)
 		{
 			try
@@ -77,20 +78,20 @@
 		#region ILoggerFactoryAdapter Members
 
 		/// <summary>
-		/// Get a ILog instance by type 
+		/// Get a ILog instance by <see cref="Type" />.
 		/// </summary>
-		/// <param name="type"></param>
-		/// <returns></returns>
+		/// <param name="type">Usually the <see cref="Type" /> of the current class.</param>
+		/// <returns>An ILog instance that will write data to <see cref="Console.Out" />.</returns>
 		public ILog GetLogger(Type type)
 		{
 			return GetLogger( type.FullName );
 		}
 
 		/// <summary>
-		/// Get a ILog instance by type name 
+		/// Get a ILog instance by name.
 		/// </summary>
-		/// <param name="name"></param>
-		/// <returns></returns>
+		/// <param name="name">Usually a <see cref="Type" />'s Name or FullName property.</param>
+		/// <returns>An ILog instance that will write data to <see cref="Console.Out" />.</returns>
 		public ILog GetLogger(string name)
 		{
 			ILog log = _logs[name] as ILog;

Modified: ibatis/trunk/cs/mapper/IBatisNet.Common/Logging/Impl/NoOpLogger.cs
URL: http://svn.apache.org/viewvc/ibatis/trunk/cs/mapper/IBatisNet.Common/Logging/Impl/NoOpLogger.cs?view=diff&rev=471478&r1=471477&r2=471478
==============================================================================
--- ibatis/trunk/cs/mapper/IBatisNet.Common/Logging/Impl/NoOpLogger.cs (original)
+++ ibatis/trunk/cs/mapper/IBatisNet.Common/Logging/Impl/NoOpLogger.cs Sun Nov  5 09:53:41 2006
@@ -2,7 +2,7 @@
 #region Apache Notice
 /*****************************************************************************
  * $Header: $
- * $Revision: $
+ * $Revision$
  * $Date$
  * 
  * iBATIS.NET Data Mapper
@@ -29,129 +29,110 @@
 namespace IBatisNet.Common.Logging.Impl
 {
 	/// <summary>
-	/// Summary description for NoOpLogger.
+	/// Silently ignores all log messages.
 	/// </summary>
 	public sealed class NoOpLogger: ILog
 	{
-		/// <summary>
-		/// A NOP implementation.
-		/// </summary>
-		public NoOpLogger()
-		{ }
-
 		#region Members of ILog
 
 		/// <summary>
-		/// 
+		/// Ignores message.
 		/// </summary>
 		/// <param name="message"></param>
 		public void Debug(object message)
 		{
-			// NOP
+			// NOP - no operation
 		}
+
 		/// <summary>
-		/// 
+		/// Ignores message.
 		/// </summary>
 		/// <param name="message"></param>
 		/// <param name="e"></param>
 		public void Debug(object message, Exception e)
 		{
-			// NOP
+			// NOP - no operation
 		}
+
 		/// <summary>
-		/// 
+		/// Ignores message.
 		/// </summary>
 		/// <param name="message"></param>
 		public void Error(object message)
 		{
-			// NOP
+			// NOP - no operation
 		}
+
 		/// <summary>
-		/// 
+		/// Ignores message.
 		/// </summary>
 		/// <param name="message"></param>
 		/// <param name="e"></param>
 		public void Error(object message, Exception e)
 		{
-			// NOP
+			// NOP - no operation
 		}
+
 		/// <summary>
-		/// 
+		/// Ignores message.
 		/// </summary>
 		/// <param name="message"></param>
 		public void Fatal(object message)
 		{
-			// NOP
+			// NOP - no operation
 		}
+
 		/// <summary>
-		/// 
+		/// Ignores message.
 		/// </summary>
 		/// <param name="message"></param>
 		/// <param name="e"></param>
 		public void Fatal(object message, Exception e)
 		{
-			// NOP
-		}
-		/// <summary>
-		/// 
-		/// </summary>
-		/// <param name="message"></param>
-		public void Info(object message)
-		{
-			// NOP
-		}
-		/// <summary>
-		/// 
-		/// </summary>
-		/// <param name="message"></param>
-		/// <param name="e"></param>
-		public void Info(object message, Exception e)
-		{
-			// NOP
+			// NOP - no operation
 		}
 
 		/// <summary>
-		/// 
+		/// Ignores message.
 		/// </summary>
 		/// <param name="message"></param>
-		public void Trace(object message)
+		public void Info(object message)
 		{
-			// NOP
+			// NOP - no operation
 		}
 
 		/// <summary>
-		/// 
+		/// Ignores message.
 		/// </summary>
 		/// <param name="message"></param>
 		/// <param name="e"></param>
-		public void Trace(object message, Exception e)
+		public void Info(object message, Exception e)
 		{
-			// NOP
+			// NOP - no operation
 		}
 
 		/// <summary>
-		/// 
+		/// Ignores message.
 		/// </summary>
 		/// <param name="message"></param>
 		public void Warn(object message)
 		{
-			// NOP
+			// NOP - no operation
 		}
 
 
 		/// <summary>
-		/// 
+		/// Ignores message.
 		/// </summary>
 		/// <param name="message"></param>
 		/// <param name="e"></param>
 		public void Warn(object message, Exception e)
 		{
-			// NOP
+			// NOP - no operation
 		}
 
-
 		/// <summary>
-		/// 
+		/// Always returns <see langword="false" />.
 		/// </summary>
 		public bool IsDebugEnabled
 		{
@@ -159,7 +140,7 @@
 		}
 
 		/// <summary>
-		/// 
+		/// Always returns <see langword="false" />.
 		/// </summary>
 		public bool IsErrorEnabled
 		{
@@ -168,7 +149,7 @@
 		}
 
 		/// <summary>
-		/// 
+		/// Always returns <see langword="false" />.
 		/// </summary>
 		public bool IsFatalEnabled
 		{
@@ -176,7 +157,7 @@
 		}
 
 		/// <summary>
-		/// 
+		/// Always returns <see langword="false" />.
 		/// </summary>
 		public bool IsInfoEnabled
 		{
@@ -184,15 +165,7 @@
 		}
 
 		/// <summary>
-		/// 
-		/// </summary>
-		public bool IsTraceEnabled
-		{
-			get { return false; }
-		}
-
-		/// <summary>
-		/// 
+		/// Always returns <see langword="false" />.
 		/// </summary>
 		public bool IsWarnEnabled
 		{
@@ -201,5 +174,4 @@
 
 		#endregion
 	}
-}
-
+}
\ No newline at end of file