Skip to content

Commit

Permalink
Small fix to C# logger (#2889)
Browse files Browse the repository at this point in the history
  • Loading branch information
bernardnormier authored Oct 11, 2024
1 parent 5a1a5b4 commit db505c5
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 14 deletions.
14 changes: 4 additions & 10 deletions csharp/src/Ice/Internal/Instance.cs
Original file line number Diff line number Diff line change
Expand Up @@ -1127,12 +1127,9 @@ public void destroy()
_initData.observer.setObserverUpdater(null);
}

if (_initData.logger is LoggerAdminLogger loggerAdminLogger)
{
LoggerAdminLogger logger = _initData.logger as LoggerAdminLogger;
if (logger != null)
{
logger.destroy();
}
loggerAdminLogger.destroy();
}

//
Expand Down Expand Up @@ -1230,12 +1227,9 @@ public void destroy()
Monitor.PulseAll(_mutex);
}

if (_initData.logger is FileLoggerI fileLogger)
{
Ice.FileLoggerI logger = _initData.logger as Ice.FileLoggerI;
if (logger != null)
{
logger.destroy();
}
fileLogger.destroy();
}
}

Expand Down
7 changes: 3 additions & 4 deletions csharp/src/Ice/Internal/LoggerAdminLoggerI.cs
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@

namespace Ice.Internal;

internal interface LoggerAdminLogger : Ice.Logger
internal interface LoggerAdminLogger : Logger
{
Ice.Object getFacet();

Expand Down Expand Up @@ -69,10 +69,9 @@ public void destroy()
Monitor.PulseAll(_mutex);
}

Ice.FileLoggerI fileLoger = _localLogger as Ice.FileLoggerI;
if (fileLoger != null)
if (_localLogger is FileLoggerI fileLogger)
{
fileLoger.destroy();
fileLogger.destroy();
}
}

Expand Down

0 comments on commit db505c5

Please sign in to comment.