Annotated class action resolver - add action bean - loop at the bottom is just 
for log.debug but is not surrouned with isDebugEnabled
-------------------------------------------------------------------------------------------------------------------------------------

                 Key: STS-677
                 URL: http://www.stripesframework.org/jira/browse/STS-677
             Project: Stripes
          Issue Type: Task
          Components: ActionBean Dispatching
    Affects Versions: Release 1.5.1
            Reporter: John Newman


Hi,

This snippet here, line 146 .. this runs all the way through and is effectively 
a noop and wasted cycles if debug is not on.  That whole block should be 
surround with is debug enabled, but the log class doesn't have such a method.

            // Print out the event mappings nicely
            for (Map.Entry<String, Method> entry : classMappings.entrySet()) {
                String event = entry.getKey();
                Method handler = entry.getValue();
                boolean isDefault = DEFAULT_HANDLER_KEY.equals(event);

                log.debug("Bound: ", clazz.getSimpleName(), ".", 
handler.getName(), "() ==> ",
                        binding, isDefault ? "" : "?" + event);
            }

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

        

------------------------------------------------------------------------------
This SF.net email is sponsored by:
High Quality Requirements in a Collaborative Environment.
Download a free trial of Rational Requirements Composer Now!
http://p.sf.net/sfu/www-ibm-com
_______________________________________________
Stripes-development mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/stripes-development

Reply via email to