Hi Lilian,

On Thu, 2005-08-25 at 10:36 -0400, Lillian Angel wrote:
> I changed this to raise an AssertionError.
> 
>      catch (BadLocationException ble)
>        {
> -        return "";
> +        throw new AssertionError("This should not happen");
>        }
>    }

Thanks and sorry to keep nitpicking. But if you do that then it is
really convenient if you chain the original exception:

  catch (BadLocationException ble)
    {
      // This should never happen.
      throw new AssertionError(ble);
    }

That way if this ever happens the original stack trace can be
recovered/shown.

Cheers,

Mark

Attachment: signature.asc
Description: This is a digitally signed message part

_______________________________________________
Classpath-patches mailing list
[email protected]
http://lists.gnu.org/mailman/listinfo/classpath-patches

Reply via email to