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