-
Notifications
You must be signed in to change notification settings - Fork 724
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
Fix fseek test #2055
Fix fseek test #2055
Conversation
This seems to break the build in an odd way, or at least my build:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I’ll approve these changes, but I noticed that the use of void main() in fseek_test.c is not ISO C compliant. Is there any reason why the contents of main() can’t be moved to a new function, which can then be called from an int main()? This wouldn't alter the current code's behavior and would make it ISO C compliant.
I didn't actually know that |
@islas Can you update the PR description to note that |
Rebased to v4.6.1 to resolve conflicts from duplicate changes |
TYPE: bug fix
KEYWORDS: fseek, compilation, cmake
SOURCE: internal
DESCRIPTION OF CHANGES:
Problem:
The fseek test lacks correct syntax causing false negative reports of feature not existing when newer compiler standards disallow this.
Solution:
Add
int
to the main program in the fseek test. Also to add further robustness in detecting this feature, use the-D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE=1
generally *nix standard defines.LIST OF MODIFIED FILES:
M CMakeLists.txt
M confcheck/CMakeLists.txt
M tools/fseek_test.c
RELEASE NOTE: Fix fseek test