Annotations on anonymous classes

Steve Sides steve.sides at
Mon Feb 18 11:46:04 PST 2013

On 2/18/2013 11:38 AM, Steve Sides wrote:
> On 2/17/2013 8:50 PM, Werner Dietl wrote:
>> I finished implementing the changed handling for anonymous classes, as
>> described by Alex in:
>> Steve/SQE: I only added one simple test to newlocations:
>> I tested that the annotations end up in the right locations using javap.
>> None of the existing test styles seemed suitable to test for
>> annotations on an anonymous class.
>> Please do expand the tests and let me know if you see any issues with 
>> this.
>> cu, WMD.
> There are some in (case4), but they are 
> currently @ignore'd.
> I have a webrev posted for tl/langtools which cleans up these tests so 
> they will be run and not ignored.
> It factors the failing cases out of test 1 so it runs completely. If 
> filters the failing cases out of test2 (too many to factor out).
> I recently added a couple more test cases to Combinations* 
> covering fields and methods.
I should have said,  if add tests cases for "fields and methods in 
anonymous classes".
> I had posted some at 
> (TestAnonInner*.java), but they were not posted as webrevs. :/
> Last week, I incorporated them into one of the Combinations tests 
> Friday, but it would be smoother if the aforementioned webrev were 
> push first.
> -steve

More information about the type-annotations-dev mailing list