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
