program crashes javac following recent updates

[Werner Dietl]

Jon, Steve,

my change definitely fixes the reported crash, but it might now put
type annotations on a ClassSymbol instead (or in addition?) of on the
correct MethodSymbol.
I think this might be happening because that code is probably visited
twice, once before and once after becoming its own top-level class.

Steve, we previously discussed having additional tests that ensure
that the Lambda conversion puts type annotations on the right Symbols.
None of the existing tests failed additionally after my change, so I'm
not sure whether I broke this or not.
Could you point us to test cases that ensure that type annotations
on/within Lambdas are converted correctly?

Thanks,
cu, WMD.

On Wed, Jul 24, 2013 at 11:53 PM, Werner Dietl <wdietl at gmail.com> wrote:
> Jon,
>
> I pushed a fix:
>
> http://hg.openjdk.java.net/type-annotations/type-annotations/langtools/rev/01303809577e
>
> Please let me know whether this looks good.
>
> Cheers,
> cu, WMD.
>
> On Wed, Jul 24, 2013 at 6:32 PM, Jonathan Gibbons
> <jonathan.gibbons at oracle.com> wrote:
>> Werner,
>>
>> Maurizio has reported that a test program has started crashing as a result
>> of a recent push from the type-annotations forest. Do you have time to take
>> a look?  You can find details here:
>>
>> http://bugs.sun.com/view_bug.do?bug_id=8020715
>>
>> -- Jon
>
>
>
> --
> http://www.google.com/profiles/wdietl



-- 
http://www.google.com/profiles/wdietl