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
Add declaration before code to guidelines #3032
Conversation
Codecov Report
@@ Coverage Diff @@
## master #3032 +/- ##
=======================================
Coverage 90.27% 90.27%
=======================================
Files 213 213
Lines 35004 35004
=======================================
Hits 31601 31601
Misses 3403 3403
Continue to review full report at Codecov.
|
Is the idea to support it from now on? |
We already have code in the repository with this, so nothing really changes. We are just documenting the guideline. |
docs/StyleGuide.md
Outdated
@@ -37,6 +37,21 @@ elaborates on the situation. Instead, we've tried our best to develop | |||
a consistent style that roughly follows the style of the PostgreSQL | |||
source code. | |||
|
|||
### Declarations before code | |||
|
|||
C99 support having code before a declaration, similar to C++, and we |
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.
support => supports
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.
Fixed.
Declarations before code has been informally supported and we have several examples in the code. This commit adds that guideline to the coding guidelines.
2b1c0e2
to
38fa10b
Compare
Declarations before code has been informally supported and we have
several examples in the code. This commit adds that guideline to the
coding guidelines.