Conversation
Ps. Doesn't really matter here since they are all private fields, but we are starting to do that for everything new (by tgross35)
|
Hi @tgross35, Could you help me understand why |
|
Hi @chengr4, thanks for finishing this! I really wanted to do this myself but I got caught up in a bunch of work for my university so I ended up forgetting about this. One thing to note, since now Also make sure to run the style checker in |
9fade92 to
d79f324
Compare
d79f324 to
37afe49
Compare
|
@highjeans Thank for answering; I totally forgot 🫠. Again, Thanks for leaving the PR for me to study this crate. Actually, I failed bad `ELAST` value at byte 0: rust: 106 (0x6a) != c 107 (0x6b)
rust bytes: 6a 00 00 00
c bytes: 6b 00 00 00
bad `HOST_VM_INFO64_COUNT` value at byte 0: rust: 38 (0x26) != c 40 (0x28)
rust bytes: 26 00 00 00
c bytes: 28 00 00 00
bad `vm_statistics64_data_t` size: rust: 152 != c 160
bad `vm_statistics64` size: rust: 152 != c 160
size of `vm_statistics64_data_t` is 160 in C and 152 in Rust
size of `struct vm_statistics64` is 160 in C and 152 in RustCan you please guide me how to evaluate and fix this bug? |
Description
This PR fixes #2903. This PR picks up the work started by @highjeans in #4336 which had stalled. I just followed up the comments from @tgross35 in the PR and tried to finish it.
Sources
Checklist
libc-test/semverhave been updated*LASTor*MAXareincluded (see #3131)
cd libc-test && cargo test --target mytarget);especially relevant for platforms that may not be checked in CI