I got a new preserve_literals related nullpointer with this version:
```
#macro (testMacro $param)
$param
#end
#testMacro({})
```
produces the following exception
Caused by: java.lang.NullPointerException
at
org.apache.velocity.runtime.parser.node.NodeUtils.tokenLiteral(NodeUtils.java:133)
at
org.apache.velocity.runtime.parser.node.SimpleNode.literal(SimpleNode.java:355)
at
org.apache.velocity.runtime.directive.VelocimacroProxy.handleArgValues(VelocimacroProxy.java:423)
at
org.apache.velocity.runtime.directive.VelocimacroProxy.render(VelocimacroProxy.java:212)
at
org.apache.velocity.runtime.directive.RuntimeMacro.render(RuntimeMacro.java:328)
at
org.apache.velocity.runtime.directive.RuntimeMacro.render(RuntimeMacro.java:258)
at
org.apache.velocity.runtime.parser.node.ASTDirective.render(ASTDirective.java:301)
at
org.apache.velocity.runtime.parser.node.SimpleNode.render(SimpleNode.java:423)
at org.apache.velocity.Template.merge(Template.java:358)
at org.apache.velocity.Template.merge(Template.java:262)
at
org.xwiki.velocity.internal.DefaultVelocityEngine.evaluate(DefaultVelocityEngine.java:271)
... 70 more
Same thing with literal arrays.
I noticed you were mentioning maps and arrays in
https://issues.apache.org/jira/browse/VELOCITY-904 but getting a
nullpointer does not feels like what you were referring to.
On Sat, Dec 28, 2019 at 2:11 PM Claude Brisson
<cla...@renegat.net.invalid> wrote:
>
> The test build of Velocity Engine 2.2 RC3 is available.
>
> No determination as to the quality ('alpha,' 'beta,' or 'GA') of
> Velocity Engine 2.1 RC3 has been made, and at this time it is simply a
> "test build". We welcome any comments you may have, and will take all
> feedback into account if a quality vote is called for this build.
>
> Release notes:
>
> *
> https://dist.apache.org/repos/dist/dev/velocity/velocity-engine/2.2/release-notes.html
>
> Distribution:
>
> * https://dist.apache.org/repos/dist/dev/velocity/velocity-engine/2.2/
>
> Maven 2 staging repository:
>
> *
> https://repository.apache.org/content/repositories/orgapachevelocity-1031/
>
> Documentation:
>
> * https://velocity.apache.org/engine/2.2/
>
> Sources:
>
> * https://svn.apache.org/repos/asf/velocity/engine/tags/2.2/
>
> Release Candidates History:
>
> * RC1 Initial RC
>
> * RC2
> - added BigInteger and BigDecimal implicit conversions
> - [VELOCITY-923] fixed a parser regression for `$foo||`
> - [VELOCITY-904] fixed two corner case bugs for the
> velocimacro.arguments.preserve_literals backward compatibility flag
> - fixed engine and dependency versions in README and mention the
> parser customization feature in the *building* section
> - nicified README links
> - upgraded surfire plugin version from 2.19.1 to 2.22.1
> - upgraded maven-jar-plugin from 3.1.1 to 3.2.2
> - added version 1.2 for extra-enforcer-rules
> - upgraded maven-javadoc-plugin from 3.1.0 to 3.1.1
> - upgraded findbugs-maven-plugin from 3.0.4 to 3.0.5
> - upgraded maven-release-plugin from *unspecified* to 3.0.0-M1
> - added a new templatized static class
> org.apache.velocity.runtime.VelocityEngineVersion.java
> - use the File Separator control character to mark the end of stream
> for the parser (instead of the zero-width space char)
> - reviewed packaging of engine examples (refreshed content, plus made
> them as a standalone zip file with readme, shell scripts, dependencies
> and examples sources rather than a meaningless standalone pom next to a
> jar without explanations...)
>
> * RC3
> - [VELOCITY-904] fixed yet another corner case bugs for the
> velocimacro.arguments.preserve_literals backward compatibility flag
> - upgraded SLF4J from 1..7.28 to 1.7.30
>
>
> --
> Claude
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscr...@velocity.apache.org
> For additional commands, e-mail: dev-h...@velocity.apache.org
>
--
Thomas Mortagne
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@velocity.apache.org
For additional commands, e-mail: dev-h...@velocity.apache.org
