[
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)