kaidokert / fixed-bigint-rs / 29694740326

98%

main: 97%

Pull #153

github

Address review on the heapless string surface

- Fix the feature-free test build (codex): FixedUInt's Display is effectively
  num-traits-gated, so requiring Carrier: Display broke --no-default-features.
  Move the Display bound and generic Display test to the num-traits harness
  (carrier_num_traits.rs); HeaplessBigInt's own feature-independent Display is
  covered in tests/heapless_string.rs.
- Simplify hex_fmt to iterate `limb.to_be_bytes()` instead of manual
  shift/mask, dropping the `u8: TryFrom<T>` bound from hex_fmt/LowerHex/UpperHex
  (gemini).
- Hoist the radix multiplier out of the from_str_radix loop (gemini).
- Broaden the from_str_radix test: leading zeros, uppercase hex, radix < 2
  (sourcery).

Kept MAX_DIGITS = 20: u64 is the widest MachineWord (u128 is only the
DoubleWord), so 20 decimal digits per limb-block always suffice, and a
per-limb `size_of::<T>() * 3` array size needs generic_const_exprs.

79 of 79 new or added lines in 3 files covered. (100.0%)

4637 of 4737 relevant lines covered (97.89%)

6149.53 hits per line