maven-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Brian Fox (JIRA)" <j...@codehaus.org>
Subject [jira] Reopened: (MENFORCER-33) Tidy up EvaluateBeanshell rule
Date Sun, 02 Mar 2008 23:39:28 GMT

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

Brian Fox reopened MENFORCER-33:
--------------------------------


> 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

        

Mime
View raw message