bindgen 0.49.* derives Copy for types containing _IncompleteArrayField

[nagisa]

Input C/C++ Header

typedef struct {
 unsigned int magic;
 unsigned char data[];
} pk;

Bindgen Invocation

    bindgen::Builder::default()
        .header("wrapper.h")
        .no_copy("(?i)mutex")
        .blacklist_type("_bindgen_ty_1")
        .opaque_type("max_align_t")
        .clang_args(nix_cflags.split(" "))

Actual Results

error[E0204]: the trait `Copy` may not be implemented for this type
     --> somewhere deep in the file
      |
69596 | #[derive(Debug, Copy, Clone)]
      |                 ^^^^
...
69599 |     pub data: __IncompleteArrayField<::std::os::raw::c_uchar>,
      |     --------------------------------------------------------- this field does not implement `Copy`

Expected Results

Successful compilation, which is what happened a couple of versions ago…

[emilio]

Hmmm... Is this with rust 1.39, and goes away with 1.38 by chance? If so, this is an issue that was fixed in #1627.

[emilio]

I can publish a 0.49 minor version if needed with that PR.

[nagisa]

This started occurring after a nightly bump so plausible that it could be a 38 to 39 change.

…

On Tue, 12 Nov 2019, 13:29 Emilio Cobos Álvarez, ***@***.***> wrote: I can publish a 0.49 minor version if needed with that PR. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#1674?email_source=notifications&email_token=AAFFZUSP5HX7LVGFIUVYSWLQTMN2LA5CNFSM4JMGLDWKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOED3744I#issuecomment-553123441>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAFFZUQDUHW6DKMCXE7UPL3QTMN2LANCNFSM4JMGLDWA> .

[emilio]

Yeah, I'm ~sure the only explanation for this is #1627, which was triggered by a rust change, see there.

If you can't update to 0.50 or 0.51 I'm happy to publish a minor version of 0.49 with that fixed, just lmk.