Skip to content

Use proper typedef for uint64_t#125

Closed
pps83 wants to merge 1 commit intopowturbo:masterfrom
pps83:u64typedef
Closed

Use proper typedef for uint64_t#125
pps83 wants to merge 1 commit intopowturbo:masterfrom
pps83:u64typedef

Conversation

@pps83
Copy link
Copy Markdown
Contributor

@pps83 pps83 commented Feb 18, 2024

Depending on order of includes using define causes errors

Depending on order of includes using define causes errors
@powturbo powturbo closed this Apr 13, 2026
@pps83
Copy link
Copy Markdown
Contributor Author

pps83 commented Apr 13, 2026

Here's what it means to have this kind of define without this fix: you cannot include stdint.h/inttypes.h after this time_.h file for obvious reasons. Those headers define uint64_t and preprocessor will replace uint64_t with unsigned __int64. This obviously will produce compilation error.

With the change, if you have a typedef, those headers will simply make identical typedef. This won't produce errors and/or warnings.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants