Johannes Verelst wrote:
I would like to propose this implementation as a HACK in a few days, so
please send your comments on this code now, to prevent having
implementation-specific comments during the hack vote.

The main 'issue' I have with the approach is that the original setup of teh dove xpath parameter went for the 'xpath' look (ok, stupid idea, sue me), and this one doesn't.
My suggestion is to abandon the xpath look, at least for the extension.
Two ways to solve this:


1. Add a new 'path' parameter. Keep the xpath parameter as is, (with its 'xpath' like syntax), but either deprecate it or note it as a possible alternate syntax.

2. alter the xpath parameter so it also accepts non-'xpath' syntaxes (which effectively means it ignores the first three chars if they are '/*@')

I favor 1 because I think it is silly to have a 'xpath' parameter that doesn't have an xpath syntax.
No further comments on the hack so far.


--
Pierre van Rooden
Mediapark, C 107 tel. +31 (0)35 6772815
"Never summon anything bigger than your head."




Reply via email to