camel-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Claus Ibsen (JIRA)" <j...@apache.org>
Subject [jira] Commented: (CAMEL-3759) After switching to slf4j, we can get rid of the 'isTraceEnabled', 'isDebugEnabled' and 'isInfoEnabled' statements
Date Sat, 05 Mar 2011 14:20:46 GMT

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

Claus Ibsen commented on CAMEL-3759:
------------------------------------

You can only use at most 2 placeholders like that. If you need 3 or more, you have to do it
the old fashion way.
Unfortunately sfl4j doesn't support varargs.


> 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, 3.0.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 is automatically generated by JIRA.
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

       

Mime
View raw message