7081804: Remove cause field from javax.xml.crypto.NoSuchMechnismException

Sean Mullan sean.mullan at oracle.com
Fri Sep 2 12:58:29 PDT 2011


On 9/2/11 1:43 AM, Sebastian Sickelmann wrote:
>>> Here is the updated webrev:
>>> http://oss-patches.24.eu/openjdk8/NoSuchMechanismException/7011804_0/
>> Hmm, the main problem I have with this change is that the printStackTrace
>> methods will no longer print the stack trace of the cause because it will always
>> be null. That doesn't seem right to me, as it could be considered an
>> incompatible change, and it will make it harder to debug issues.
> The printStackTrace in Throwable calls the overridden getCause().
> Maybe we should add @Override to it.
> Updated the webrev to:
> http://oss-patches.24.eu/openjdk8/NoSuchMechanismException/7011804_1/

In that case, my main concern is addressed then. I would probably want someone
from our TCK team to also review it with respect to JSR 105 compatibility, so
I'll see if I can find someone.

But first, can you expand your webrev to include the other Exception classes in
javax.xml.crypto.**?

Thanks,
Sean



More information about the security-dev mailing list