Log4net
  1. Log4net
  2. LOG4NET-97

Make Hierarchy's ILoggerFactory aware of the repository's LevelMap

    Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Trivial Trivial
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.2.11
    • Component/s: None
    • Labels:
      None

      Description

      Instead of creating the RootLogger with Level.Debug:

      return new RootLogger(Level.Debug);

      shouldn't the DefaultLoggerFactory use:

      return new RootLogger(repository.LevelMap.LookupWithDefault(Level.Debug));

      in case the user has remapped Debug to another value.

        Activity

        Ron Grabowski created issue -
        Hide
        Ron Grabowski added a comment -

        Wrap usages of static Level levels with LevelMap.LookupWithDefault.

        Show
        Ron Grabowski added a comment - Wrap usages of static Level levels with LevelMap.LookupWithDefault.
        Ron Grabowski made changes -
        Field Original Value New Value
        Attachment Make_ILoggerFactory_use_LevelMap.patch [ 12342297 ]
        Hide
        Nicko Cadell added a comment -

        I have committed this patch.

        Show
        Nicko Cadell added a comment - I have committed this patch.
        Nicko Cadell made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Fix Version/s 1.2.11 [ 12310980 ]
        Assignee Nicko Cadell [ nicko ]
        Resolution Fixed [ 1 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        152d 16h 39m 1 Nicko Cadell 06/Mar/07 20:01

          People

          • Assignee:
            Nicko Cadell
            Reporter:
            Ron Grabowski
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development