add defmt impls for core::ptr::NonNull and fn(...) -> Ret#753
add defmt impls for core::ptr::NonNull and fn(...) -> Ret#753bors[bot] merged 3 commits intoknurling-rs:mainfrom Crzyrndm:ptr-nonnull
core::ptr::NonNull and fn(...) -> Ret#753Conversation
Urhengulas
left a comment
There was a problem hiding this comment.
Thank you for your contribution @Crzyrndm.
Note: not quite sure where to add tests for the output?
Please add some log statements just before following line line and run cargo xtask test-snapshot --overwrite from the repository root:
defmt/firmware/qemu/src/bin/log.rs
Line 735 in d58633f
Please also add this PR to the CHANGELOG.md.
|
Test prints added for NonNull, and fn pointers of 0/1/5 parameters for Rust/C/unsafe (formatting makes 12 a bit much, it's already noisy enough as is and the limit is about as arbitrary as it gets). |
|
Thank you! bors r+ |
|
Build succeeded! The publicly hosted instance of bors-ng is deprecated and will go away soon. If you want to self-host your own instance, instructions are here. If you want to switch to GitHub's built-in merge queue, visit their help page. |
Fixes #752
NonNull and fn pointers formatted like a raw pointer. This matches the behaviour of
Debugfor these types