[ http://jira.codehaus.org/browse/DROOLS-487?page=all ]
Greg Barton closed DROOLS-487:
------------------------------
Resolution: Fixed
Cool. Mind if I spelunk the code for other similar issues?
> Remove some inefficient exception handling
> -------------------------------------------
>
> Key: DROOLS-487
> URL: http://jira.codehaus.org/browse/DROOLS-487
> Project: drools
> Type: Improvement
> Components: io
> Versions: 2.5
> Reporter: Greg Barton
> Priority: Minor
> Fix For: 2.5
> Attachments: patch.txt
>
>
> Using exceptions for flow control can be very inefficient on most modern VMs.
> This is because exception handling is rarely optimized. Thus it is highly
> preferable to use conditionals to test for exceptional conditions and avoid
> them instead of using a try/catch to knowingly trigger them.
> I saw some code in org.drools.io.RuleSetReader.resolveSchema() that tested
> various locations for the xsd file using exceptions for flow control.
> Attached is a patch that eliminates that with identical functionality.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see:
http://www.atlassian.com/software/jira
