[LOG4NET-81] LoggerRepositorySkeleton's OnConfigurationChanged method always raises its event with EventArgs.Empty instead of passing through its EventArgs parameter.

• Type: Bug
• Status: Resolved
• Priority: Trivial
• Resolution: Fixed
• Affects Version/s: 1.2.10
• Fix Version/s: 1.2.11
• Assignee: liu ron
• Reporter: liu ron
• Created: 14/Jul/06 10:03
• Updated: 14/Jul/06 10:07
• Resolved: 14/Jul/06 10:07

Description

This code:

protected virtual void OnConfigurationChanged(EventArgs e)
{
if (e == null)

{ e = EventArgs.Empty; }

LoggerRepositoryConfigurationChangedEventHandler handler = m_configurationChangedEvent;
if (handler != null)
{ handler(this, EventArgs.Empty); }
}

should be changed to match the other event handlers in the class:

protected virtual void OnConfigurationChanged(EventArgs e)
{
if (e == null)
{ e = EventArgs.Empty; }

LoggerRepositoryConfigurationChangedEventHandler handler = m_configurationChangedEvent;
if (handler != null)

{ handler(this, e); // do not always pass in EventArgs.Empty }

}