-
Notifications
You must be signed in to change notification settings - Fork 2.1k
timex: macro names are misleading #4211
Copy link
Copy link
Closed
Labels
Discussion: RFCThe issue/PR is used as a discussion starting point about the item of the issue/PRThe issue/PR is used as a discussion starting point about the item of the issue/PRType: enhancementThe issue suggests enhanceable parts / The PR enhances parts of the codebase / documentationThe issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation
Description
Am I the only one that thinks the name of SEC_IN_USEC, SEC_IN_MS, MS_IN_USEC, and USEC_IN_NS are somewhat missleading?
- They are inconsistent in itself. Some abbreviate "seconds" with
SEC, some withS - Their values reflect the opposite of their name: there are 1000000 microseconds in a second, not 1000000 seconds in a microsecond; there are 1000 milliseconds in a second, not 1000 seconds in a microsecond; etc.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
Discussion: RFCThe issue/PR is used as a discussion starting point about the item of the issue/PRThe issue/PR is used as a discussion starting point about the item of the issue/PRType: enhancementThe issue suggests enhanceable parts / The PR enhances parts of the codebase / documentationThe issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation