Skip to content

Conversation

@Caellian
Copy link

  • Added no_std support with optional std feature
    • Migrated from std to core and alloc for no_std compatibility
    • Replaced dbg! macro calls with feature-gated versions for no_std compatibility
  • Feature-gated language modules for selective compilation
  • Default implementations for format_and_value() and format_decimal_and_value() in LangInterpreter trait
    • Removed duplicate format_and_value() and format_decimal_and_value() implementations from language modules
  • Optimized parsing a bit
    • Previous code converted DigitString to String by prepending zeroes, then parsed the longer String
    • New code parses DigitString directly. It's infallible because only numbers are inserted by the library.
      • DigitString should be pub(crate) and not exposed by the library for this to be 100% true.
  • Applied fixes for clippy warnings
  • Some other very minor tweaks

- Added `no_std` support with optional `std` feature
    - Migrated from `std` to `core` and `alloc` for `no_std` compatibility
    - Replaced `dbg!` macro calls with feature-gated versions for `no_std` compatibility
- Feature-gated language modules for selective compilation
- Default implementations for `format_and_value()` and `format_decimal_and_value()` in `LangInterpreter` trait
    - Removed duplicate `format_and_value()` and `format_decimal_and_value()` implementations from language modules
- Optimized parsing a bit
    - Previous code converted `DigitString` to `String` by prepending zeroes, then parsed the longer `String`
    - New code parses `DigitString` directly. It's infallible because only numbers are inserted by the library.
        - `DigitString` should be `pub(crate)` and not exposed by the library for this to be 100% true.
- Applied fixes for clippy warnings
- Some other very minor tweaks

Signed-off-by: Tin Švagelj <tin.svagelj@live.com>
Signed-off-by: Tin Švagelj <tin.svagelj@live.com>
- Add two tests which check for something I got wrong while developing
  to avoid regression in future.

Signed-off-by: Tin Švagelj <tin.svagelj@live.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant