Nullus157 / bs58-rs

Another Rust Base58 codec implementation
Apache License 2.0
75 stars 24 forks source link

encode: implement Display for EncodeBuilder #97

Open mina86 opened 1 year ago

mina86 commented 1 year ago

Add implementation of core::fmt::Display for EncodeBuilder which tries to avoid memory allocations when encoding sensibly small values (specifically, shorter than 96 bytes).

codecov[bot] commented 1 year ago

Codecov Report

Patch coverage: 100.00% and project coverage change: +0.99 :tada:

Comparison is base (be42edf) 72.91% compared to head (99b2e6d) 73.91%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #97 +/- ## ========================================== + Coverage 72.91% 73.91% +0.99% ========================================== Files 4 4 Lines 288 299 +11 ========================================== + Hits 210 221 +11 Misses 78 78 ``` | [Impacted Files](https://app.codecov.io/gh/Nullus157/bs58-rs/pull/97?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Nullus157) | Coverage Δ | | |---|---|---| | [src/encode.rs](https://app.codecov.io/gh/Nullus157/bs58-rs/pull/97?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Nullus157#diff-c3JjL2VuY29kZS5ycw==) | `83.08% <100.00%> (+1.48%)` | :arrow_up: |

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

mina86 commented 1 year ago

Friendly ping.

mina86 commented 1 year ago

@Nemo157, friendly ping