Re: [8u-dev] Request for Review and Approval to backport: 8160518: Semicolon is not recognized as comment starting character (Kerberos)

2016-07-09 Thread Sean Coffey 

Looks fine. Approved for jdk8u-dev.

regards,
Sean.

On 08/07/2016 18:04, Ivan Gerasimov wrote:

Hello!

I'd like to backport this fix into jdk8u-dev.
The fix is essentially the same as in jdk9, but could not be used 
verbatim because of the code derivation.


Bug: https://bugs.openjdk.java.net/browse/JDK-8160518
Jdk9 change: http://hg.openjdk.java.net/jdk9/dev/jdk/rev/4d5c6e8bad2d
Jdk9 review: 
http://mail.openjdk.java.net/pipermail/security-dev/2016-July/014377.html

Jdk8 webrev: http://cr.openjdk.java.net/~igerasim/8160518/01/webrev/

Would you please help review it and approve the backport?

With kind regards,
Ivan

[8u-dev] Request for Review and Approval to backport: 8160518: Semicolon is not recognized as comment starting character (Kerberos)

2016-07-08 Thread Ivan Gerasimov 

Hello!

I'd like to backport this fix into jdk8u-dev.
The fix is essentially the same as in jdk9, but could not be used 
verbatim because of the code derivation.


Bug: https://bugs.openjdk.java.net/browse/JDK-8160518
Jdk9 change: http://hg.openjdk.java.net/jdk9/dev/jdk/rev/4d5c6e8bad2d
Jdk9 review: 
http://mail.openjdk.java.net/pipermail/security-dev/2016-July/014377.html

Jdk8 webrev: http://cr.openjdk.java.net/~igerasim/8160518/01/webrev/

Would you please help review it and approve the backport?

With kind regards,
Ivan