Optional wider range of base unit exponents? about units

[phlptp]

for 1, it would need to be done in C++11 constexpr expression which could be difficult, and actually the wrapping behavior is utilized in a few cases so not sure I would want that checking in general. Something like a checked_multiply might be appropriate that wouldn't operate in constexpr and does any required checks might be appropriate.

As far as increasing the range the space requirements for many of our applications mean we didn't really consider that. I think most of the code would work fine though, Obviously have to change the unit_data class and the static asserts since they would all fail, but most other stuff should be the same. It would create some gaps in the data structures but if space isn't much of an issue that shouldn't be too big of deal. As far as a PR, I would want to be careful with it but not completely opposed. We would need to add one of the CI builds that actually used the change to make sure all the tests still passed.

[SimonHeybrock]

Something like a checked_multiply might be appropriate that wouldn't operate in constexpr and does any required checks might be appropriate.

That sounds like a good suggestion, I'll look into it.

I think most of the code would work fine though, Obviously have to change the unit_data class and the static asserts since they would all fail, but most other stuff should be the same.

It does, I had experimented with this change yesterday and didn't encounter real issues so far. But you make a very good point about the CI and test coverage.

[SimonHeybrock]

Fixed in #142.