units icon indicating copy to clipboard operation
units copied to clipboard

Renaming the units to their natural spelling

Open JohelEGP opened this issue 7 years ago • 3 comments

For example, renaming meter_t to meters. This has probably been discussed in various places. I'll link to them as I find them.

To answer https://github.com/nholthaus/units/pull/150#issuecomment-405940556, I believe that, in the long run, renaming the units to their natural spelling is the right way to go. The _t suffix already has some uses that don't match the use in this library's units, like in the the standard type traits and the arguably more similar use in the types of <cstdint>. As the library becomes more popular, the _t suffix will become more of a baggage than the natural spelling the library can exploit with its CTAD capabilities and further possible improvements.

JohelEGP avatar Jul 18 '18 17:07 JohelEGP

I agree with the logic. v3.0 is going to have a hefty upgrade burden on the users from 2.3, but that's the price of progress. We'll need an upgrade guide in the docs.

nholthaus avatar Jul 18 '18 19:07 nholthaus

What should we do about https://github.com/nholthaus/units/blob/5483589322daad285eedd4e2822058ba4e6640d8/include/units/pressure.h#L75 and similar cases which have the same singular and plural name?

JohelEGP avatar Oct 06 '18 21:10 JohelEGP

dimensionless_unit also needs a name revision.

JohelEGP avatar Oct 06 '18 22:10 JohelEGP