-
Notifications
You must be signed in to change notification settings - Fork 146
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
GNU -Wall Warning Reduction #1049
base: main
Are you sure you want to change the base?
GNU -Wall Warning Reduction #1049
Conversation
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.
All good changes. A note on this one. Perhaps the GNU compiler is not up-to-date, but:
Warn when a source code line will be truncated
The line length limit in Fortran 2023 is ten thousand characters (10,000 characters). For continued lines (using &
) the limit is one million characters (1,000,000 characters). In other words, there's probably nowhere in the UFS code that passes the Fortran line length limits. Only the most complicated cpp-generated code risks hitting the line length limits. Alas, we are not using 2030s-era compilers, so we are stuck with the Fortran 2018 limits.
…nt at (1) [-Wampersand]'
…ed at (1) not set [-Wreturn-type]"
c27ffe3
to
64d33d5
Compare
This should also be targeted at ufs-community:ufs/dev |
Reducing GNU
-Wall
warnings per issue #495.Commits fix the following warning flags implied in
-Wall