commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Simone Tripodi <>
Subject Re: [ognl] Create parser dynamically
Date Sat, 11 Jun 2011 16:49:23 GMT
Hi Jochen!!!
green light from my side, that would be a *great* improvement IMHO!!!
Targeting to a different package would at the same time help you
detecting the differences with current code, so filling missing parts
should be less painful.
Thanks a lot in advance for your help and feel free to ping me
directly if any help is needed!
All the best, have a nice weekend!!!

On Sat, Jun 11, 2011 at 5:58 PM, Gary Gregory <> wrote:
> On Jun 11, 2011, at 11:49, Jochen Wiedmann <> wrote:
>> Hi,
>> I have looked into the old OgnlParser sources. My intention was to
>> detect the changes that have been made manually. If that would be
>> possible, I'd basically reapply those changes to ognl.jjt, or ognl.jj
>> and then would switch to a dynamically generated parser.
>> Unfortunately, it is almost impossible for me to detect such changes.
>> For example, in the commit from 2008-01-25 22:17:10, I really cannot
>> distinguish between changes that are due to the parser upgrade and
>> other changes.
>> As a consequence, all I can offer is to change the build in a way that
>> the parser is now generated dynamically up to a point where the test
>> suite runs fine. However, that is obviously not a guarantee for
>> compatibilty. I can also offer further maintenance, if
>> incompatibilties are detected and if they can be fixed by changes in
>> the parser.
>> If we'd do that, I'd also like to move the parser and it's
>> accompanying files (almost everything in o.a.ognl) to a separate
>> package. IMO, we should clearly separate between public API and
>> implementation. Fortunately, the change of package and Maven id's is a
>> good opportunity for that.
>> WDYT?
> Sounds good. Something like a parser should be generated from the
> grammar at build time. The trick is to still make the generate source
> available in source zip.
> Maybe a package name with impl or gen or internal would make it
> obvious not to muck with these sources.
> Gary
>> Jochen
>> --
>> Capitalism is the astounding belief that the most wickedest of men
>> will do the most wickedest of things for the greatest good of
>> everyone.
>> John Maynard Keynes (
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail:
>> For additional commands, e-mail:
> ---------------------------------------------------------------------
> To unsubscribe, e-mail:
> For additional commands, e-mail:

To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message