Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

lib517: fix C89 constant signedness #9572

Closed
wants to merge 1 commit into from
Closed

Conversation

monnerat
Copy link
Contributor

@monnerat monnerat commented Sep 22, 2022

In C89, positive integer literals that overflow an int but not an unsigned int may be understood as a negative int.

lib517.c:129:3: warning: this decimal constant is unsigned only in ISO C90
   {"Sun, 06 Nov 2044 08:49:37 GMT", 2362034977 },
   ^

In C89, positive integer literals that overflow an int but not an
unsigned int may be understood as a negative int.

lib517.c:129:3: warning: this decimal constant is unsigned only in ISO C90
   {"Sun, 06 Nov 2044 08:49:37 GMT", 2362034977 },
   ^
bagder
bagder approved these changes Sep 22, 2022
@bagder
Copy link
Member

bagder commented Sep 22, 2022

Thanks!

@bagder bagder closed this in 549032d Sep 22, 2022
@monnerat monnerat deleted the c89-lib517 branch Sep 22, 2022
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.

None yet

2 participants