Re: [digester] assertions throw RuntimeException not Error

[robert burrell donkin <ro...@blueyonder.co.uk>]

committed. many thanks.

i'm not too sure whether i like PluginAssertionFailure (as a name) or not 
so i've left it as it is (for now). if anyone else has an opinion, then 
now's a good time to speak up...

- robert

On Wednesday, October 29, 2003, at 12:56 AM, Simon Kitching wrote:

> Hi,
>
> As agreed on the list, here is a patch to change the "assertions" in
> plugins to throw an object subclassing RuntimeException rather than
> Error.
>
> File plugins/PluginAssertionError.java can now be deleted.
> It has been replaced by PluginAssertionFailure (attached).
>
> I decided not to name the thrown class PluginAssertionException because
> this patch actually shows the disadvantage of naming the class by its
> inherited type! If you prefer, feel free to
>    s/PluginAssertionFailure/PluginAssertionException/
> in all relevant files.
>
> Regards,
>
> Simon
>   ---------------------------------------------------------------------
> To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org
> For additional commands, e-mail: commons-dev-help@jakarta.apache.org


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