commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From bugzi...@apache.org
Subject DO NOT REPLY [Bug 24369] - Validator.validate() exception handling for ValidatorException
Date Mon, 10 Nov 2003 19:32:07 GMT
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG 
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://nagoya.apache.org/bugzilla/show_bug.cgi?id=24369>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND 
INSERTED IN THE BUG DATABASE.

http://nagoya.apache.org/bugzilla/show_bug.cgi?id=24369

Validator.validate() exception handling for ValidatorException





------- Additional Comments From dgraham@apache.org  2003-11-10 19:32 -------
I agree with the proposed approach but it is an incompatible behavior change.  
The problem is that Validator was initially written without much regard to 
RuntimeExceptions.  We have done much to improve the situation but there may be 
places in the code where RuntimExceptions are known to be thrown and caught by 
the generic "catch Exception" block to be translated as a failure.  For example, 
a piece of code was throwing NullPointerException that wasn't noticed because of 
a "catch Exception" block.  After discovering and fixing this, performance 
improved by 50%.  IMO, a thorough analysis of the exception situation in 
Validator is needed before making this change.

---------------------------------------------------------------------
To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: commons-dev-help@jakarta.apache.org


Mime
View raw message