RFR 8063087: policytool reports error message with prefix of "java.lang.Exception"

[Xuelei Fan]

On 11/11/2014 1:31 PM, Wang Weijun wrote:
> A subclass of Exception would also return true when using instanceof, where I think the class name is still informational.
> 
Got it.  But Exception is still informational as it means it is not an
other exception.

I'm not sure it is something we really want to address.  The coding
logic does not looks reasonable or friendly.  Or, would you please add
some comments about why you want to treat the exception differently?

Thanks,
Xuelei

> --Max
> 
>> On Nov 11, 2014, at 11:14, Xuelei Fan <xuelei.fan at oracle.com> wrote:
>>
>> Why not use "instanceof"?
>>
>> Xuelei
>>
>> On 11/11/2014 10:58 AM, Wang Weijun wrote:
>>> Please review the fix at
>>>
>>>   http://cr.openjdk.java.net/~weijun/8063087/webrev.00/
>>>
>>> which is simply
>>>
>>>     void displayErrorDialog(Window w, Throwable t) {
>>>         if (t instanceof NoDisplayException) {
>>>             return;
>>>         }
>>>
>>> +        if (t.getClass() == Exception.class) {
>>> +            displayErrorDialog(w, t.getLocalizedMessage());
>>> +        } else {
>>>         displayErrorDialog(w, t.toString());
>>> +        }
>>>     }
>>>
>>> Error messages like "java.lang.Exception:No Policy Entry selected" is now "No Policy Entry selected". For other exception types, there will be no change.
>>>
>>> Thanks
>>> Max
>>>
>>
>