[ 
http://jira.codehaus.org/browse/MENFORCER-33?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Brian Fox closed MENFORCER-33.
------------------------------

       Resolution: Fixed
    Fix Version/s: 1.0

> Tidy up EvaluateBeanshell rule
> ------------------------------
>
>                 Key: MENFORCER-33
>                 URL: http://jira.codehaus.org/browse/MENFORCER-33
>             Project: Maven 2.x Enforcer Plugin
>          Issue Type: Task
>          Components: Standard Rules
>    Affects Versions: 1.0-alpha-3
>            Reporter: Benjamin Bentmann
>            Assignee: Brian Fox
>            Priority: Trivial
>             Fix For: 1.0
>
>         Attachments: eval-beanshell.patch
>
>
> The rule contains various local variables that are only used in some log 
> statements and hence seem superfluos. If logging is the sole purpose of these 
> lines, they should be enclosed in a block that is guarded by 
> {{log.isDebugEnabled()}}. There is no need to slow the check or risk failures 
> just for the fun of debug logging.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: 
http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to