• Library crate doesn't need a Cargo.lock file
• lib: Use assert_eq in necessary places
• rustfmt and clippy run
• Add gitignore to ignore Cargo.lock file
• Implement Debug for testing struct

Unaligned accesses are UB in LLVM (and silently corrupt data on at least some ARM chips), so before doing unsafe transmutes on the memory, we need to ensure the incoming slice is aligned correctly for the output type.

Add some tests to ensure this happens ... and assume the u8 arrays are 32bit aligned, so we need to drop the first element to get an unaligned u32. This may not actually be true (and if so, the test will be flaky) but the test passes for several local runs on x86-64.

https://docs.rs/crate/zero/0.1.2/source/src/lib.rs

It doesn't check the alignment as it is on GitHub: https://github.com/nrc/zero/blob/4a7247507cdc82d8018af48d7bf94b5c951a1eba/src/lib.rs#L51-L59

Maybe publish a new version?

Wasn't sure whether to file this as an issue in this crate or in your xmas-elf crate, but I chose here because the lowest-level change should be made here.

In the read_str() function, there is the potential to panic on the expect statement. When parsing a file (such as an ELF file using your xmas-elf crate), some symbols may not be encoded as UTF-8, so it'd be nice to be able to skip them and recover gracefully through a returned Result, rather than simply panicking and aborting the entire parse thread.

If you're okay with that change, I'm happy to submit a pull request to both this and the xmas-elf crate.

Also, thanks a lot for this and the xmas-elf crate, and also for making the change for my previous issue with the elf flags. It's really saved me a lot of time!!