Skip to content

Commit f203ecc

Browse files
committed
Shadowing is not enabled by default, update doc accordingly.
1 parent a26280b commit f203ecc

File tree

1 file changed

+3
-4
lines changed

1 file changed

+3
-4
lines changed

doc/developer-notes.md

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -343,10 +343,9 @@ Strings and formatting
343343
Variable names
344344
--------------
345345

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

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

0 commit comments

Comments
 (0)