fix a few issues preventing no-std builds from ... building

this includes a `Makefile` that exercises the various crate configs.
most annoyingly, several doc comments needed to grow
`#[cfg(feature="fmt")]` blocks so docs continue to build with that
feature enabled or disabled.

carved out a way to run exhaustive tests; they should be written as
`#[ignore]`, and then the makefile will run even ignored tests on the
expectation that this will run the exhaustive (but slower) suite.
exhaustive tests are not yet written. they'll probably involve spanning
4 byte sequences from 0 to 2^32-1.

1 files changed, 18 insertions, 1 deletions

diff --git a/src/lib.rs b/src/lib.rs
index b8bd4f7..9825fd2 100644
--- a/src/lib.rs
+++ b/src/lib.rs
@@ -12,6 +12,7 @@
 //!
 //! let inst = decoder.decode_slice(&[0x33, 0xc0]).unwrap();
 //!
+//! #[cfg(features="fmt")]
 //! assert_eq!("xor eax, eax", inst.to_string());
 //! ```
 //!
@@ -31,13 +32,16 @@
 //!
 //! let inst = decoder.decode_slice(&[0x33, 0x01]).unwrap();
 //!
+//! #[cfg(features="fmt")]
 //! assert_eq!("xor eax, dword [rcx]", inst.to_string());
 //!
 //! assert_eq!(Operand::Register(RegSpec::eax()), inst.operand(0));
+//! #[cfg(features="fmt")]
 //! assert_eq!("eax", inst.operand(0).to_string());
 //! assert_eq!(Operand::RegDeref(RegSpec::rcx()), inst.operand(1));
 //!
 //! // an operand in isolation does not know the size of memory it references, if any
+//! #[cfg(features="fmt")]
 //! assert_eq!("[rcx]", inst.operand(1).to_string());
 //!
 //! // and for memory operands, the size must be read from the instruction itself:
@@ -45,6 +49,7 @@
 //! assert_eq!("dword", mem_size.size_name());
 //!
 //! // `MemoryAccessSize::size_name()` is how its `Display` impl works, as well:
+//! #[cfg(features="fmt")]
 //! assert_eq!("dword", mem_size.to_string());
 //! ```
 //!
@@ -54,12 +59,23 @@
 //! mod decoder {
 //!     use yaxpeax_arch::{Arch, AddressDisplay, Decoder, Reader, ReaderBuilder};
 //!
+//!     // have to play some games so this example works right even without `fmt` enabled!
+//!     #[cfg(feature="fmt")]
+//!     trait InstBound: std::fmt::Display {}
+//!     #[cfg(not(feature="fmt"))]
+//!     trait InstBound {}
+//!
+//!     #[cfg(feature="fmt")]
+//!     impl <T: std::fmt::Display> InstBound for T {}
+//!     #[cfg(not(feature="fmt"))]
+//!     impl <T> InstBound for T {}
+//!
 //!     pub fn decode_stream<
 //!         'data,
 //!         A: yaxpeax_arch::Arch,
 //!         U: ReaderBuilder<A::Address, A::Word>,
 //!     >(data: U) where
-//!         A::Instruction: std::fmt::Display,
+//!         A::Instruction: InstBound,
 //!     {
 //!         let mut reader = ReaderBuilder::read_from(data);
 //!         let mut address: A::Address = reader.total_offset();
@@ -69,6 +85,7 @@
 //!         loop {
 //!             match decode_res {
 //!                 Ok(ref inst) => {
+//!                     #[cfg(feature="fmt")]
 //!                     println!("{}: {}", address.show(), inst);
 //!                     decode_res = decoder.decode(&mut reader);
 //!                     address = reader.total_offset();