Details

    • Type: Improvement Improvement
    • Status: Open
    • Priority: Minor Minor
    • Resolution: Unresolved
    • Affects Version/s: BSF-2.4
    • Fix Version/s: None
    • Component/s: BSF-2.x
    • Labels:
      None

      Description

      BSFManager seems to be a bit inconsistent in the way that it uses logging.

      For example, in some places is uses println() to report errors, and others it uses log.debug(), when it should probably use log.warn().

      Probably worth taking a general look at how logging is used throughout BSF.

        Activity

        Hide
        Sebb added a comment -

        Just had a look at BSFManager in trunk, and it creates a logger in the constructor.

        Seems to me that it would make more sense to create the logger as a private static final field that can then be used by static methods as well.

        Show
        Sebb added a comment - Just had a look at BSFManager in trunk, and it creates a logger in the constructor. Seems to me that it would make more sense to create the logger as a private static final field that can then be used by static methods as well.

          People

          • Assignee:
            Unassigned
            Reporter:
            Sebb
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:

              Development