[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [cp-patches]: FYI: JPasswordField
From: |
Mark Wielaard |
Subject: |
Re: [cp-patches]: FYI: JPasswordField |
Date: |
Thu, 25 Aug 2005 16:50:45 +0200 |
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
signature.asc
Description: This is a digitally signed message part