RFR: 7903674: jextract should ignore non-enum constants inside enum [v2]

Athijegannathan Sundararajan sundar at openjdk.org
Tue Feb 20 13:23:06 UTC 2024


On Tue, 20 Feb 2024 12:42:39 GMT, Maurizio Cimadamore <mcimadamore at openjdk.org> wrote:

> I believe that at least test for something like this:
> 
> ```
> enum __attribute((deprecated)) Foo {
>     CFByteOrderUnknown,
> }; 
> ```
> 
> Should be added. Windows uses a different attribute syntax (`[[deprecated]]`). So either we use a macro to define the attribute, or we just test on Linux/Mac.
> 
> It might also be useful to add tests for:
> 
> ```
> struct __attribute((deprecated))  Foo {
>    int x;
> };
> ```
> 
> e.g. struct field case. And:
> 
> ```
> typedef __attribute((deprecated)) struct  { int x } BAR;
> ```
> 
> For the nested struct case.

Will add a test

-------------

PR Comment: https://git.openjdk.org/jextract/pull/221#issuecomment-1954203701


More information about the jextract-dev mailing list