[ 
https://issues.apache.org/jira/browse/MAILET-33?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12984622#action_12984622
 ] 

Stefano Bagnara commented on MAILET-33:
---------------------------------------

This is the cause of my doubts. The stringformat one gives us more performance 
(expecially with debug strings when debug is not enabled), on the other side we 
have the current log with no level and a simple string (so we already have a 
very KISS solution). On the other side (the KISS) we could add an 
isLogEnabled(Level level) method and still let people doing something only when 
debug is enabled.

Maybe most users are more used to the non-varargs solution and the 
isLogEnabled(Level) approach, don't know.

As we moved to java5 I wanted to raise this possible solution, but this was 
just something on the table, not something I "push".

Any other opinion?

> MailetContext should provide a log method which allow to specify the loglevel
> -----------------------------------------------------------------------------
>
>                 Key: MAILET-33
>                 URL: https://issues.apache.org/jira/browse/MAILET-33
>             Project: Mailet
>          Issue Type: Improvement
>          Components: Core API
>    Affects Versions: 2.4
>            Reporter: Norman Maurer
>            Assignee: Norman Maurer
>             Fix For: 2.5
>
>
> Currently MailetContext.log(..) does not allow to specify the loglevel to use 
> for logging. We should mark this method as @deprecated and add a new one 
> which takes a  logging level argument.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to