[ 
https://issues.apache.org/jira/browse/CAMEL-3759?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15434245#comment-15434245
 ] 

ASF GitHub Bot commented on CAMEL-3759:
---------------------------------------

GitHub user YoshikiHigo opened a pull request:

    https://github.com/apache/camel/pull/1134

    Followup to CAMEL-3759

    This commit is a small followup to CAMEL-3759, which was fixed in 
aa6423972f17042af05d7f0c692756f2be60b1e3.
    
    Yoshiki, Shinpei, Hideaki, and Mei.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/YoshikiHigo/camel fix01

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/camel/pull/1134.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #1134
    
----
commit 9535a1058699da259a49a0472b25fc257c7e2e91
Author: Yoshiki Higo <[email protected]>
Date:   2016-08-24T06:12:04Z

    Followup to CAMEL-3759

----


> After switching to slf4j, we can get rid of the 'isTraceEnabled', 
> 'isDebugEnabled' and 'isInfoEnabled' statements
> -----------------------------------------------------------------------------------------------------------------
>
>                 Key: CAMEL-3759
>                 URL: https://issues.apache.org/jira/browse/CAMEL-3759
>             Project: Camel
>          Issue Type: Improvement
>    Affects Versions: 2.6.0
>            Reporter: Christian Müller
>            Assignee: Christian Müller
>             Fix For: 2.8.0
>
>
> we can get rid of the 'isTraceEnabled', 'isDebugEnabled' and 'isInfoEnabled' 
> statements with slf4j and use
> {code}
> logger.debug("Temperature set to {}. Old temperature was {}.", t, oldT);
> {code}
> instead
> christian-muellers-macbook-pro:camel cmueller$ egrep -r 
> 'isTraceEnabled|isDebugEnabled|isInfoEnabled]' . | wc -l
>     1485



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to