-
Notifications
You must be signed in to change notification settings - Fork 29
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
Consider single return per statement #190
Comments
Put a halt on this due to rapidly increasing cyclomatic complexity. If you replace the early returns with a status message that gets returned at the end, you must add a ton of 'if' statements checking if the function should be returned. |
We may want to revisit now that we've reduced complexity once (Mainly because we still have to revisit on the KMC side once open source containers are released). |
Originally layout made sense. Making issue for internal discussion of having just one return per function, perhaps utilizing gotoreturn labels. ISsue to be discussed / closed / modified as appropriate.
The text was updated successfully, but these errors were encountered: