[ https://issues.apache.org/jira/browse/MYFACES-3497?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13228577#comment-13228577 ]
Leonardo Uribe commented on MYFACES-3497: ----------------------------------------- * created methodExpressions are already cached at TagAttributeImpl. No need to do something additional in this part * it is reasonable to cache that result, because once calculated it will not change. * Suggestions are welcome. Note the same trick is used in other places like javax.faces.event.MethodExpressionActionListener. Checking this one, I notice in EL 2.2 there exists a new method called boolean isParmetersProvided() (the spell bug is intentional). > [perf] Improve EventHandler > --------------------------- > > Key: MYFACES-3497 > URL: https://issues.apache.org/jira/browse/MYFACES-3497 > Project: MyFaces Core > Issue Type: Improvement > Reporter: Martin Kočí > Priority: Minor > > org.apache.myfaces.view.facelets.tag.jsf.core.EventHandler > * cache created MethodExpressions (same as in MYFACES-3160) > * if f:event type="" is literal, cache resulting class > * review implementation of > EventHandler.Listener.processEvent(ComponentSystemEvent): uses "void" El > method invocation to detect if method exists - spec requires this but maybe > better solution exists > * .... -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira