Re: svn commit: r1753422 - /velocity/engine/trunk/velocity-engine-core/src/test/java/org/apache/velocity/test/MacroAutoReloadTestCase.java

2016-07-19 Thread Claude Brisson
I usually try to do it. Sorry if I forgot for some of them. But for this one, that's a fix for a problem I discovered while coding (the testcase needed an "mvn clean" to work). The testcase itself, added in commit 1753201 yesterday, contains a reference to the issue. Not every commit refers t

Re: svn commit: r1753422 - /velocity/engine/trunk/velocity-engine-core/src/test/java/org/apache/velocity/test/MacroAutoReloadTestCase.java

2016-07-19 Thread Sergiu Dumitriu
It is recommended to include the jira issue id in the commit message, so that jira can find the right commits for an issue, and a commit can quickly identify the relevant discussion. On Jul 19, 2016 16:27, wrote: > Author: cbrisson > Date: Tue Jul 19 20:26:58 2016 > New Revision: 1753422 > > URL

[jira] [Commented] (VELOCITY-799) Optionally add context to itself

2016-07-19 Thread Claude Brisson (JIRA)
[ https://issues.apache.org/jira/browse/VELOCITY-799?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15383808#comment-15383808 ] Claude Brisson commented on VELOCITY-799: - You are right, the spec says: In a

[jira] [Resolved] (VELOCITY-815) MapGetExecutor is very slow [PATCH]

2016-07-19 Thread Claude Brisson (JIRA)
[ https://issues.apache.org/jira/browse/VELOCITY-815?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Claude Brisson resolved VELOCITY-815. - Resolution: Fixed Assignee: Claude Brisson Fix Version/s: 2.x Patch appl