RFR (XS) JDK-8051002: Incorrectly merged share/vm/classfile/classFileParser.cpp was pushed to 8u20

[harold seigel]

Hi Lois,

The change looks good.

Harold
On 8/22/2014 2:14 PM, Lois Foltan wrote:
> Hi Everyone,
>
> Please review the following fix targeted specifically for 8u.
>
> Webrev:
>     http://cr.openjdk.java.net/~lfoltan/bug_jdk8051002/
>
> Bug: Incorrectly merged share/vm/classfile/classFileParser.cpp was 
> pushed to 8u20
>     https://bugs.openjdk.java.net/browse/JDK-8051002
>
> Summary of fix:
> Incorrect JDK 8u merge of classfile/classFileParser.cpp resulted in a 
> duplicate call to check the attribute_byte_length within 
> ClassFileParser::parse_classfile_bootstrap_methods_attribute(). The 
> first call did not reflect the fixed condition check and resulted in 
> the failure of 
> test/runtime/classFileParserBug/TestEmptyBootstrapMethodsAttr.java. 
> This change fixes the condition check and removes the duplicate call 
> to guarantee_property.
>
> Testing:
>     vm.quick.testlist, Hotspot's full jtreg tests
>
> Thanks,
> Lois
>
>
>