Commons Logging
  1. Commons Logging
  2. LOGGING-92

LogFactoryImpl javadoc says default NoOpLog, but really SimpleLog

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 1.0.1
    • Fix Version/s: 1.0.3
    • Labels:
      None
    • Environment:

      Operating System: other
      Platform: Other

      Description

      The javadoc for the "LogFactoryImpl" class says that the default logger is
      "NoOpLog". However, the code clearly shows that it uses "SimpleLog", not
      "NoOpLog" (changed by costin in revision 1.4). I prefer it falling back to
      SimpleLog, so I consider this a documentation bug, not a code bug.

        Activity

        David M. Karr created issue -
        Hide
        Craig McClanahan added a comment -

        Fixed in nightly build 20021123. I updated the remaining documentation
        references to reflect the actual functionality that SimpleLog is the default.

        Show
        Craig McClanahan added a comment - Fixed in nightly build 20021123. I updated the remaining documentation references to reflect the actual functionality that SimpleLog is the default.
        Henri Yandell made changes -
        Field Original Value New Value
        issue.field.bugzillaimportkey 14571 12340420
        Henri Yandell made changes -
        Component/s Logging [ 12311124 ]
        Affects Version/s Nightly Builds [ 12311648 ]
        Key COM-269 LOGGING-92
        Assignee Jakarta Commons Developers Mailing List [ commons-dev@jakarta.apache.org ]
        Project Commons [ 12310458 ] Commons Logging [ 12310484 ]
        Henri Yandell made changes -
        Affects Version/s Nightly Builds [ 12311790 ]
        Henri Yandell made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Dennis Lundberg made changes -
        Fix Version/s 1.0.3 [ 12311839 ]
        Affects Version/s 1.0.1 [ 12311824 ]
        Affects Version/s Nightly Builds [ 12311790 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Resolved Resolved Closed Closed
        1874d 2h 30m 1 Henri Yandell 02/Jan/08 07:48

          People

          • Assignee:
            Unassigned
            Reporter:
            David M. Karr
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development