Camel
  1. Camel
  2. CAMEL-3759

After switching to slf4j, we can get rid of the 'isTraceEnabled', 'isDebugEnabled' and 'isInfoEnabled' statements

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.6.0
    • Fix Version/s: 2.8.0
    • Component/s: None
    • Labels:
      None

      Description

      we can get rid of the 'isTraceEnabled', 'isDebugEnabled' and 'isInfoEnabled' statements with slf4j and use

      logger.debug("Temperature set to {}. Old temperature was {}.", t, oldT);
      

      instead

      christian-muellers-macbook-pro:camel cmueller$ egrep -r 'isTraceEnabled|isDebugEnabled|isInfoEnabled]' . | wc -l
      1485

        Issue Links

          Activity

          Christian Müller created issue -
          Christian Müller made changes -
          Field Original Value New Value
          Link This issue is related to CAMEL-2229 [ CAMEL-2229 ]
          Christian Müller made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Fix Version/s 3.0.0 [ 12315691 ]
          Resolution Fixed [ 1 ]
          Christian Müller made changes -
          Status Resolved [ 5 ] Closed [ 6 ]

            People

            • Assignee:
              Christian Müller
              Reporter:
              Christian Müller
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development