[ http://issues.apache.org/jira/browse/VELOCITY-503?page=comments#action_12454101 ] Henning Schmiedehausen commented on VELOCITY-503: -------------------------------------------------
Will moved that out in http://svn.apache.org/viewvc?view=rev&revision=149378. Will, you also removed the test from the if branch of the (icd != null...) check starting in line 168; did you intentionally leave the reevaluation in the second branch or did you just forget to remove it (if yes, we can probably remove the loop safely as suggested by Claude. Nice finding anyway though. Thanks a lot! > Method arguments are evaluated twice > ------------------------------------ > > Key: VELOCITY-503 > URL: http://issues.apache.org/jira/browse/VELOCITY-503 > Project: Velocity > Issue Type: Bug > Components: Engine > Affects Versions: 1.5 beta2 > Environment: all > Reporter: Claude Brisson > Priority: Minor > Fix For: 1.5 > > Attachments: fix-arg-eval.patch > > > Method arguments are evaluated twice in ASTMethod - no apparent reason. -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://issues.apache.org/jira/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
