Darren Kulp

Results 43 comments of Darren Kulp

> It's not quite the same. I was not clear. I think the title of this issue ("Make derive Debug configurable for specific structures") and the original description ("We would...

Hmm, I am not sure we can do this. ```bash cargo clean cargo build cargo run ``` results in: ``` error: target `bindgen` in package `bindgen` requires the features: `clap`...

Thanks for the report, @Dushistov. In your first example: if you put an extra set of parentheses around the pattern, resulting in `((C4(?!ReplicatorActivity)(.+))|(FLError))`, does it start working as you expect?...

Also #2195 may be relevant, if it shows that this problem is biting users.

I took the liberty of rebasing upon `master` at ~~c27578fac5372575e3caa933a83eac931ae3b53f~~ 74b38670b04067598ce488a455725b324bafdaee and fixing up the failing tests. If CI passes, this might be ready for the next major release, since...

> I think changing those tests to use `.*` and landing this should be ok @emilio Now each commit in this branch independently passes tests. I assume this should wait...

> we think it's pretty important to know if this change would be worth it from the maintainers point of view or if a less intrusive way to solve the...

This passes `cargo test` on my M1 Mac with LLVM 13, but not CI. I expect to investigate when I have time.

As of 4575fab9e7b491742d4f6de31f828b6bff0e5396 there is one CI job passing all tests with Clang 9.0. Right now this requires a lot of hacks to `build.rs`, but it gives hope.

This might be, sadly, just a case of the STL not being supported very well by `bindgen`. See https://github.com/rust-lang/rust-bindgen/issues/1808#issuecomment-647197424 and https://github.com/rust-lang/rust-bindgen/issues/1597#issuecomment-526871534.