Boost C++ Libraries: Ticket #10089: constexpr support in units

anonymous — Tue, 03 Jun 2014 11:30:49 GMT

I've submitted a pull-request here: github.com/boostorg/units/pull/1

owner changed

Jürgen Hunold — Wed, 05 Nov 2014 14:20:33 GMT

owner changed from Matthias Schabel to Jürgen Hunold

lou_langholtz@… — Fri, 18 Nov 2016 00:56:54 GMT

Boost 1.62.0 doesn't appear to support constexpr in the units library yet this request was opened 2 years ago. Why the delay in supporting constexpr in the units library? What can I do to help this along?

gonzalobg88@… — Fri, 18 Nov 2016 09:32:11 GMT

This feature would be nice to have, but nobody has had time to do it "with Boost quality" yet.

I wrote a patch back then, but it was very rough, introduced some problems, and I didn't have time to work on it anymore.

You can check the pull-request on github, go through the comments/discussion, and try to improve it.

If you need help, I can give you feedback.

lou_langholtz@… — Sun, 20 Nov 2016 00:26:31 GMT

Replying to gonzalobg88@…:

This feature would be nice to have, but nobody has had time to do it "with Boost quality" yet.

Does this assessment include the pull request by muggenhor on Aug 14? https://github.com/boostorg/units/pull/22

...You can check the pull-request on github, go through the comments/discussion, and try to improve it.

Thank you. Started looking on github and found muggenhor's request that I linked above. Looks like his pull request is open while yours got closed. There is no one assigned to muggenhor's request nor labels/milestone however.

anonymous — Mon, 30 Jan 2017 03:50:28 GMT

I don't see any comments on the PR, what needs to be done to get it merged?

anonymous — Wed, 10 Jan 2018 20:39:45 GMT

This would be really nice to have this. Without it, you can't specify array sizes based on math done with "units". ex, I can't do this:

constexpr quantity<time> secs_4 = 4*seconds; constexpr quantity<time> secs_2 = 2*seconds; constexpr int array_size = static_cast<int>(secs_4/secs_2); int an_array[array_size];

I also converting everything to "static const" instead of constexpr, but that didn't work either.