Re: cvs commit: jakarta-velocity/src/java/org/apache/velocity Context.java

[Jon Stevens]

on 12/18/2000 11:40 AM, "Jose  Alberto Fernandez" <[EMAIL PROTECTED]>
wrote:

> Hey guys,
> 
> I have a problem with this commit. This change in signature of the
> context methods means that now we will have to catch exceptions all over the
> code.
> 
> Also, the Hashtable is already doing all this checks. It is much more
> efficient to catch the NPE and then figure out why it happened.
> Since it will hardly ever happened.
> 
> In any case, we should be throwing some RuntimeException not an
> indiscriminate Exception. We should define our own Exception class:
> 
> class ContextException extends RuntimeException
> {
> ...
> }
> 
> Jose Alberto

Yea yea yea...I'm going to fix it.

-jon

-- 
Honk if you love peace and quiet.