Hi Chris,
thanks for pointing - I forgot to add copyright. Will fix.

Regarding jar attachment - this is a slightly modified version of test as received from customer. If there is any mechanism allowing to use maven artifact (I am not aware of it) with jprt, it would be definitelly better solution, but I didn't find it.

I could rewrite the test that it would test just not adding this handler, but it wouldn't explain much what the issue was about - I could write it to javadoc?

Let me know, what you think.

Thanks
Miran

On 19/05/14 19:27, Chris Hegarty wrote:
Miran,

The source change looks ok to me. Have you commented out the line adding the handler because it may be reinstated, or as a reminder that it was deliberately omitted?

The test, at the very minimum, will need a suitable copyright header. I also wonder if there is any other way of testing this issue without adding, what appears to be, a large jar file.

-Chris.

On 19/05/14 14:36, Miroslav Kos wrote:
Hi,
this is review request for:

8043129: JAF initialisation in SAAJ clashing with the one in javax.mail

JBS: https://bugs.openjdk.java.net/browse/JDK-8043129
WEBREV:
http://cr.openjdk.java.net/~mkos/8043129/jaxws.00/
http://cr.openjdk.java.net/~mkos/8043129/jdk.00/ (test)

This solves conflict between JAF handlers for JAX-WS and javax.mail. The
issue has been introduced in the latest CPU.

Thanks
Miran

Reply via email to