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.

Sign up for GitHub

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

No impact: return statement cleanup #724

Open

scrasmussen wants to merge 1 commit into NCAR:main from scrasmussen:no-impact/return-statement-cleanup

Member

scrasmussen commented Oct 18, 2023

TYPE: no impact

KEYWORDS: code clean up

SOURCE: Soren Rasmussen, NCAR

DESCRIPTION OF CHANGES: Code cleanup: removal of redundant 'return' statements before an 'end subroutine' statement

TESTS CONDUCTED: Built and ran with Croton to double check. Shouldn't change functionality at all

Checklist

Closes issue #xxxx (An issue must exist or be created to be closed. The issue describes and documents the problem and general solution, the PR describes the technical details of the solution.)
Tests added (unit tests and/or regression/integration tests)
Backwards compatible
Requires new files? If so, how to generate them.
Documentation included
Short description in the Development section of NEWS.md


Cleanup: removal of redundant 'return' statements before an 'end subr…

a02be7a

…outine' statement

scrasmussen force-pushed the no-impact/return-statement-cleanup branch from e43143c to a02be7a Compare

June 5, 2024 16:02

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