https://issues.apache.org/bugzilla/show_bug.cgi?id=57441

--- Comment #2 from Mark Thomas <ma...@apache.org> ---
I think it is time to ditch Jasper's partial EL parser and use
o.a.el.lang.ExpressionBuilder.createNode() to fully parse the EL with a
'proper' EL implementation and then process those nodes.

The down side is that I think this would be the first thing to make Jasper
depend on Tomcat's EL implementation rather than any EL implementation. Users
would still be free to use another EL impl at runtime - Jasper would just
always use Tomcat's impl directly so it can access the parsed nodes. (I need to
check the dependency issues. The poms already show a dependency here.)

The upside should be an end to the steady stream of bugs with every new major
release as the various new EL features are found to break the simple EL parsing
Jasper does. A side benefit is deleting more code and I'm always happy when I
can provide the same or better functionality with less code.

-- 
You are receiving this mail because:
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to