fixes to javap/typeAnnotations tests

Steve Sides steve.sides at oracle.com
Fri Mar 1 11:32:10 PST 2013


yep.  I have a test for anon/inner class but it's hard to fix it with 
the current repeating annotation bugs.
I'll make tests for the bugs I reported, but the larger test may have to 
either wait or be chopped up.
-steve

On 3/1/2013 11:17 AM, Werner Dietl wrote:
> Hi Steve,
>
> thanks for this patch! I pushed it to type-annotations:
>
> http://hg.openjdk.java.net/type-annotations/type-annotations/langtools/rev/cc7184b9aa57
>
> Could you also provide patches for the 4 bug reports you recently sent?
> (Messages "AssertionError at
> com.sun.tools.javac.code.Type$AnnotatedType.<init>" and "some new
> bugs").
> Working on the fix for the first bug nearly undid the fix for the
> second. It would be easier to keep everything working if all tests
> were in the repository.
>
> Thanks,
> cu, WMD.
>
>
> On Thu, Feb 28, 2013 at 11:32 PM, Steve Sides<steve.sides at oracle.com>  wrote:
>> Hi,
>> Here are some changes to the tests in tools/javap/typeAnnotations which
>> allow the failing tests to pass.
>> http://cr.openjdk.java.net/~ssides/8009256/
>> They are changes for the move of type annotations to the code attributes and
>> a little clean up here and there.
>> The change of the test name in Presence.java to TestPresence was keep it
>> from being overwritten by the other test which use 'Test.java'.
>> I forgot to put it back to 'Test.java' but it's benign.
>>
>> -steve
>
>



More information about the type-annotations-dev mailing list