classpath-patches
[Top][All Lists]
Advanced

[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

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


reply via email to

[Prev in Thread] Current Thread [Next in Thread]