Browse Source

Shadowing is not enabled by default, update doc accordingly.

tags/v0.15.1
Pavel Janík 4 years ago
parent
commit
f203ecc621
1 changed files with 3 additions and 4 deletions
  1. 3
    4
      doc/developer-notes.md

+ 3
- 4
doc/developer-notes.md View File

@@ -343,10 +343,9 @@ Strings and formatting
Variable names
--------------

The shadowing warning (`-Wshadow`) is enabled by default. It prevents issues rising
from using a different variable with the same name.

Please name variables so that their names do not shadow variables defined in the source code.
Although the shadowing warning (`-Wshadow`) is not enabled by default (it prevents issues rising
from using a different variable with the same name),
please name variables so that their names do not shadow variables defined in the source code.

E.g. in member initializers, prepend `_` to the argument name shadowing the
member name:

Loading…
Cancel
Save