RFR JDK-8040292: The verifier does not check that some annotation attributes are illegally specified more than once

[harold seigel]

Hi Coleen,

Thanks for the review.  I agree with your refactoring comment.

Harold

On 4/23/2014 7:00 PM, Coleen Phillimore wrote:
>
> This change looks good.  I think this code needs to be refactored 
> longer term.
> Coleen
>
> On 4/22/14, 9:21 AM, harold seigel wrote:
>> Hi,
>>
>> Please review this relatively small JDK-9 fix for bug JDK-8040292. 
>> The JVM Specification states that there can only be one instance of 
>> certain attributes.  The VM checked this for some, but not all 
>> cases.  This fix adds the missing checks.
>>
>> Open webrev:  http://cr.openjdk.java.net/~hseigel/bug_8040292_9/
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8040292
>>
>> The fix was tested with the Hotspot JTREG tests, JCK Lang tests, nsk 
>> Quick tests, and the test case in this webrev.
>>
>> Thanks! Harold
>