[foreign] RFR 8219442: jextract does not generate InnerClasses attribute properly
Sundararajan Athijegannathan
sundararajan.athijegannathan at oracle.com
Thu Feb 21 15:39:58 UTC 2019
Thanks.
I've added FIXME comments for both cases. For macros, we don't yet use
the corresponding JType "jt" yet. I'm going ahead with pushing.
Webrev for the record:
https://cr.openjdk.java.net/~sundar/8219442/webrev.01/
-Sundar
On 21/02/19, 8:47 PM, Maurizio Cimadamore wrote:
> Very nice and pragmatic fix; also very nice test.
>
> Only two nits:
>
> * the logic you have will not work for more nested types (e.g. A.B.C),
> since we don't represent nested class types with recursive ClassType
> chains. But (as discussed offline) we're good for now, jextract
> doesn't emit stuff like this.
>
> * I see there's no call to visitinner for macro types; again this is
> ok for now, but could bite back as soon as we start to support proper
> pointer constants (pointers are generic).
>
> Maurizio
>
> On 21/02/2019 15:00, Sundararajan Athijegannathan wrote:
>> Please review.
>>
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8219442
>> Webrev: https://cr.openjdk.java.net/~sundar/8219442/webrev.00/
>>
>> Thanks,
>> -Sundar
More information about the panama-dev
mailing list