Why is inccorect Static Method called?

[Greif]

I am trying to call a static method within a sealed class. However, the wrong
method keeps getting called instead.

In a seperate class I am calling the following code:
LogHelper.Log(" Sample Exception", ex, LogLevel.ERROR) ;

Here is the weird thing, the public static void Log(string message, LogLevel
level) method is always called instead of the public static void Log(string
message, Exception e, LogLevel level) method. I don't know why. The method
signature is correct. I am stumped. Any help would be greatly appreciated.
Thanks.

Here is my class definition:

private static readonly ILog m_Log;
private object m_MessageToLog;
private Exception m_ExceptionToLo g;
private CallerInfo m_CallerInfo;
private LogLevel m_LogLevelToLog ;

static LogHelper()
{
m_Log = LogManager.GetL ogger("appLogge r");
}

private LogHelper(objec t message, CallerInfo callerInfo, LogLevel level)
{
this.m_MessageT oLog = message;
this.m_LogLevel ToLog = level;
this.m_CallerIn fo = callerInfo;
}

private LogHelper(objec t message, Exception e, CallerInfo callerInfo,
LogLevel level)
{
this.m_MessageT oLog = message;
this.m_LogLevel ToLog = level;
this.m_Exceptio nToLog = e;
this.m_CallerIn fo = callerInfo;
}

public static void Log(string message, LogLevel level)
{
LogX(message, (Exception)null , level);
}

public static void Log(string message, Exception e, LogLevel level)
{
LogX(message, e, level);
}

private static void LogX(string message, Exception e, LogLevel level)
{
StackFrame sf;
CallerInfo callerInfo;
LogHelper logger;
Thread loggerThread;

// Grab the data from 2 frames up the stack (source application)
sf = new StackFrame(2, true);

// Populate the caller info
callerInfo = new CallerInfo(sf.G etFileName(),
sf.GetMethod(). ReflectedType.N ame, sf.GetMethod(). Name,
sf.GetFileLineN umber(), sf.GetFileColum nNumber());

// Create an instance of the logger
logger = new LogHelper(messa ge, e, callerInfo, level);

// Create the worker thread
loggerThread = new Thread(new ThreadStart(log ger.writeToLog) );

// Give the thread a name
loggerThread.Na me = "Log4Net Worker Thread";

// Set the thread to the background
loggerThread.Is Background = true;

// Start the thread
loggerThread.St art();
}

private void writeToLog()
{
//Use Mapped Diagnostic Context to set the source information
MDC.Set(LOGGER_ FILE_NAME, m_CallerInfo.Fi leName);
MDC.Set(LOGGER_ CLASS_NAME, m_CallerInfo.Cl assName);
MDC.Set(LOGGER_ METHOD_NAME, m_CallerInfo.Me thodName);
MDC.Set(LOGGER_ LINE_NUMBER, m_CallerInfo.Li neNumber.ToStri ng());
MDC.Set(LOGGER_ COLUMN_NUMBER, m_CallerInfo.Co lumnNumber.ToSt ring());
if(!(m_Exceptio nToLog==null))
MDC.Set(LOGGER_ MESSAGE, m_ExceptionToLo g.Message);

switch(m_LogLev elToLog)
{
case LogLevel.DEBUG:
if (m_Log.IsDebugE nabled)
{
if (m_ExceptionToL og != null)
{
m_Log.Debug(m_M essageToLog, m_ExceptionToLo g);
}
else
{
m_Log.Debug(m_M essageToLog);
}
}
break;
case LogLevel.ERROR:
if (m_Log.IsErrorE nabled)
{
if (m_ExceptionToL og != null)
{
m_Log.Error(m_M essageToLog, m_ExceptionToLo g);
}
else
{
m_Log.Error(m_M essageToLog);
}
}
break;
case LogLevel.FATAL:
if (m_Log.IsFatalE nabled)
{
if (m_ExceptionToL og != null)
{
m_Log.Fatal(m_M essageToLog, m_ExceptionToLo g);
}
else
{
m_Log.Fatal(m_M essageToLog);
}
}
break;
case LogLevel.INFO:
if (m_Log.IsInfoEn abled)
{
if (m_ExceptionToL og != null)
{
m_Log.Info(m_Me ssageToLog, m_ExceptionToLo g);
}
else
{
m_Log.Info(m_Me ssageToLog);
}
}
break;
case LogLevel.WARN:
if (m_Log.IsWarnEn abled)
{
if (m_ExceptionToL og != null)
{
m_Log.Warn(m_Me ssageToLog, m_ExceptionToLo g);
}
else
{
m_Log.Warn(m_Me ssageToLog);
}
}
break;
default:
break;
}
}

[Jon Skeet [C# MVP]]

Greif <Gr***@discussi ons.microsoft.c om> wrote:

I am trying to call a static method within a sealed class. However, the wrong
method keeps getting called instead.

In a seperate class I am calling the following code:
LogHelper.Log(" Sample Exception", ex, LogLevel.ERROR) ;

Here is the weird thing, the public static void Log(string message, LogLevel
level) method is always called instead of the public static void Log(string
message, Exception e, LogLevel level) method. I don't know why. The method
signature is correct. I am stumped. Any help would be greatly appreciated.

It sounds very unlikely, to be honest.

Could you post a short but complete program which demonstrates the
problem?

See http://www.pobox.com/~skeet/csharp/complete.html for details of
what I mean by that.

--
Jon Skeet - <sk***@pobox.co m>
http://www.pobox.com/~skeet
If replying to the group, please do not mail me too