Utilize `#` specifier to implement `Display` trait for SI unit

[EFanZh]

So that it is possible to use {} for decimal units (KB, MB, GB), and {:#} for binary units (KiB, MiB, GiB).

[wookietreiber]

This seems rather ad-hoc to me. I would prefer explicitly naming the unit rather than having this be done ad-hoc with formatting specifiers.

[loyd]

KB by default looks strange to me. Never have I met a programmer thinking in decimal units instead of binary units.

[MrCroxx]

KB by default looks strange to me. Never have I met a programmer thinking in decimal units instead of binary units.

+1 for this.

Seem the repo is not maintained for some time. I forked this repo to https://github.com/foyer-rs/bytesize and made some differences.

Differences from the original bytesize

• Use SI format by default with Display.
• Use "KiB" for SI unit.

Considering the changes, the version of foyer-bytesize crate starts from "2" to differ from the original bytesize crate.

FYI: https://github.com/foyer-rs/bytesize#differences-from-the-original-bytesize

If you need the features, you can use foyer-bytesize as:

bytesize = { package = "foyer-bytesize", version = "2", features = ["serde"]}

And I'm happy to either help to maintain the repo, or contribute back to the upstream if the author is still maintaining this repo.

[MrCroxx]

Hi, folks. I am currently responsible for advancing the development of bytesize with two other developers. In the next release version, bytesize will use IEC units by default.

[loyd]

@MrCroxx
Thank you for taking maintenance (it was unreliable to depend on forks for my OOS crates).

(Not important) note about SI: according to the standard, "k" should be used instead of "K": https://en.wikipedia.org/wiki/Kilo-
So, if you're making breaking changes, it should probably be revised.

Also, the current version of the code confuses IEC (Ki, Mi, Gi, etc.) and SI (k, M, G, etc.) everywhere (so "SI" in the code is actually "IEC" and vice versa)

[MrCroxx]

(Not important) note about SI: according to the standard, "k" should be used instead of "K": en.wikipedia.org/wiki/Kilo-
So, if you're making breaking changes, it should probably be revised.

Also, the current version of the code confuses IEC (Ki, Mi, Gi, etc.) and SI (k, M, G, etc.) everywhere (so "SI" in the code is actually "IEC" and vice versa)

Hi, @loyd . Thank you for reminding me. Just checked the wikis and I'll fix them these days. 🙌

[robjtede]

I think the headline of this issue is generally considered to be less preferable than ideas in #32, so I'm closing it.