Use `uint16_t` instead of `uint8_t` for semver tokens

Neargye / semver

Semantic Versioning for modern C++

MIT License

296 stars 32 forks source link

Use `uint16_t` instead of `uint8_t` for semver tokens #39

Closed TylerJang27 closed 1 year ago

TylerJang27 commented 1 year ago

Use uint16_t to expand valid semver to range from 0.0.0 to 65535.65535.65535 (rather than to 255.255.255). This also applies to prerelease version numbers.

This was a problem for tools like ruff, which is now at 0.0.270, as well as circleci, which is now at v0.1.26837. Using 2 bytes to represent each of the semver tokens should resolve this problem, as well as most other tools.

Neargye commented 1 year ago

Thanks!

Neargye commented 1 year ago

In general, can immediately uint32_t