Am 09.09.2011 15:24, schrieb Sebastian Sickelmann:
Am 08.09.2011 19:47, schrieb Sebastian Sickelmann:
Am 07.09.2011 19:51, schrieb Sean Mullan:
On 9/3/11 1:04 PM, Sebastian Sickelmann wrote:
Am 02.09.2011 21:58, schrieb Sean Mullan:
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.
Fine, that would be good.
But first, can you expand your webrev to include the other Exception classes in
javax.xml.crypto.**?
The new webrev is here:
http://oss-patches.24.eu/openjdk8/NoSuchMechanismException/7011804_2/
In some classes the initCause comment is misspelled as initCaus. What about a test case, for example testing to make sure initCause throws an exception? Can
you write one?
While creating an test(suggested code below) for this, i thought a little bit about if it is really good to change the behavoir of the ctors without a cause (defaultctor, ctor with message). What is the best behavoir (see DEFAULT and MESSAGE_ONLY cases below) . Should in the cases DEFAULT and MESSAGE_ONLY the cause mutable? I actually think this would the better solution, cause it is what the users can actually do with the exceptions in javax/xml/crypto. Or should the test check on imutability in all cases?

Also, I have asked someone from the TCK team to look at this and he said he will do that by Friday. It might require a CCC change because the behavior of initCause is different. I am hoping it doesn't require a JSR 105 maintenance
revision though.

--Sean



i have updated the webrev [0].
But i think that L69 and L72 of the test should be changed to checkMutable and the implementation of the exceptions accordantly.

[0] http://oss-patches.24.eu/openjdk8/NoSuchMechanismException/7011804_3

-- Sebastian
Any comments / progress on this?
-- Sebastian

Reply via email to