rust-embedded / cortex-m-semihosting

Semihosting for ARM Cortex-M processors

Apache License 2.0

40 stars 19 forks source link

Closed nickray closed 5 years ago

nickray commented 5 years ago

I'm sorry, it's me again...

std::dbg panics on failure, given dbg! is advertised as "quick n' dirty" I think this behaviour makes sense here too (no need to type .unwrap() everywhere, no unused Result warnings)
also to be more analogous, actually use heprintln (as already advertised in documentation)
I went ahead and added the nod to 2018 edition missing last time

rust-highfive commented 5 years ago

r? @thejpster

(rust_highfive has picked a reviewer for you, use r? to override)

bors[bot] commented 5 years ago

Build succeeded