Skip to content

Commit 5049f7f

Browse files
committed
Merge #14625: Make clear function argument case in dev notes
9605bbd Make clear function argument case in dev notes (Carl Dong) Pull request description: Rationale: For new developers, they might be confused if they see that function arguments are sometimes `camelCase`'d in the codebase. This makes it clear that they _should_ be `snake_case`'d (maybe because no one's gotten to fixing them yet). Tree-SHA512: 9db16d1fedf9761121844a0865ae3fefea94b5dbdfb36cb18f99cbc73e117f7d798a019f28a1c8bca19772502de2f9ed063f03bd911ffc4d248ec7386cd87d97
2 parents f69d922 + 9605bbd commit 5049f7f

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

doc/developer-notes.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -69,7 +69,7 @@ tool to clean up patches automatically before submission.
6969
- **Symbol naming conventions**. These are preferred in new code, but are not
7070
required when doing so would need changes to significant pieces of existing
7171
code.
72-
- Variable and namespace names are all lowercase, and may use `_` to
72+
- Variable (including function arguments) and namespace names are all lowercase, and may use `_` to
7373
separate words (snake_case).
7474
- Class member variables have a `m_` prefix.
7575
- Global variables have a `g_` prefix.

0 commit comments

Comments
 (0)