RFR : (bf) Backout 7190219 fix for jdk7u (JCK issue)

[Jonathan Lu]

Hi Sean,

Alan has got a fix for the behavior change introduced by 7190219, if it 
solves the JCK failure, is it possible to back port 7199551 as well 
instead of dropping this defect-fixing patch 7190219?

Change set for 7199551,
http://hg.openjdk.java.net/jdk8/tl/jdk/rev/3cfb621d5e7e

Best regards
Jonathan

On 10/06/2012 01:10 AM, Seán Coffey wrote:
> Jonathan,
>
> I'm looking to back this fix out of jdk7u-dev since it's causing a JCK 
> failure. The JCK test itself is too strict but we should play safe and 
> revert behaviour for 7u now until the JCK issue is resolved.
>
> There's a JCK bug following the JCK test issue  (7199569)
>
> ===
> JCK test 
> api/java_nio/CharBuffer/index.html#Methods2[appendCharSequence0006]
> assumes that ReadOnlyBufferException has some priority over 
> BufferOverflowException if an application attempts to put a String 
> into a read-only CharBuffer that has insufficient space. The spec does 
> not explicitly state, which exception should be thrown if several 
> constraints are violated at the same time, so the test should be 
> fixed/relaxed.
> ====
>
> Here's the backout webrev : (obtained with hg backout (& interrupted))
> http://cr.openjdk.java.net/~coffeys/webrev.8000178/ 
> <http://cr.openjdk.java.net/%7Ecoffeys/webrev.8000178/>
>
> JDK bug report : 8000178 (not visible yet)
>
> ==========
> The fix for 7190219 causes a JCK failure.
> api/java_nio/CharBuffer/index.html#Methods2[appendCharSequence0006]
>
> That JCK test in itself is too strict and is under review via Bug ID 
> 7199569. For the purpose of update release though, we should withdraw 
> the 7190219 fix.
> ==========
>
> regards,
> Sean.

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.openjdk.java.net/pipermail/nio-dev/attachments/20121008/c800060a/attachment.html