i've managed to pass positional information through from the parser to
the Command and Test objects. my implementation uses ScriptCoordinate
objects which contain details of line and column numbers. when a
problem occurs during the evaluation of Command and Test
implementations, this object is used to construct SyntaxException
instances with positional information.

it was necessary to add ScriptCoordinate to ExecutableCommand#execute
and ExecutableTest#execute as well as to objects in the inheritance
hierarchy (AbstractCommand, AbstractTest and so on).

this means breaking backwards compatibility

it also means that altering the current pure, clean API design in a
pragmatic (but inelegant) fashion. probably the original APIs are
superior and quite possibly once we start down this route, more
pragmatic changes will be needed later.

so, i've prepared a patch so that people can take a look before i
commit: https://issues.apache.org/jira/browse/JAMES-780.

opinions?

- robert

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to