* Otis Gospodnetic
| I finally had the chance/had to look at QueryParser.jj.
| I see what you are talking about.
| 
| I'll add TokenMgrError to the throws clause (I don't know JavaCC that
| well, so please let me know if this is wrong) and to the Javadoc.

Why is a subclass of java.lang.Error thrown? Isn't it better to throw an
Exception instead? An Error signals that this is a _very_ serious
problem and applications should generally not try to catch it. See:

http://java.sun.com/j2se/1.3/docs/api/java/lang/Error.html

It would be much better to catch the error inside the query parser and
rethrow it as a ParseException.

Geir O.


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

Reply via email to