Bug 24302 – rename the variable "degrees" in sumtype.d at least in the kelvin case

Status
NEW
Severity
normal
Priority
P1
Component
phobos
Product
D
Version
D2
Platform
All
OS
All
Creation time
2023-12-26T20:19:43Z
Last change time
2024-12-01T16:42:09Z
Assigned to
No Owner
Creator
kdevel
Moved to GitHub: phobos#10540 →

Comments

Comment #0 by kdevel — 2023-12-26T20:19:43Z
src/phobos/std/sumtype.d ```d struct Fahrenheit { double degrees; } struct Celsius { double degrees; } struct Kelvin { double degrees; } ``` According to WP there is no "degree Kelvin" ("°K") [1]. I would like to suggest renaming "degrees" to "temperature" or "value". The "°" has been dropped 56 (fifty-six) years ago [2]. Also note that the name of the unit is "kelvin" (all lower case). [1] https://en.wikipedia.org/wiki/Kelvin [2] https://www.bipm.org/en/committees/cg/cgpm/13-1967/resolution-3
Comment #1 by robert.schadek — 2024-12-01T16:42:09Z
THIS ISSUE HAS BEEN MOVED TO GITHUB https://github.com/dlang/phobos/issues/10540 DO NOT COMMENT HERE ANYMORE, NOBODY WILL SEE IT, THIS ISSUE HAS BEEN MOVED TO GITHUB