Skip to content

Commit 1502fb5

Browse files
committed
Minor additions related to compilation flags
1 parent fdccf35 commit 1502fb5

1 file changed

Lines changed: 8 additions & 2 deletions

File tree

README.md

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -623,7 +623,12 @@ Use tools
623623
are inevitably going to be broken.
624624

625625
- Compile and lint your code and files with all warnings enabled and treat
626-
warnings as errors, otherwise they are useless.
626+
warnings as errors, otherwise they are useless. There can be exceptions for
627+
specific warnings of course. Counterarguments like “-Werror Introduces a
628+
Toolchain Dependency”
629+
(https://embeddedartistry.com/blog/2017/05/22/werror-is-not-your-friend/) are
630+
weak, since the real problem there is that the language standard is not
631+
enforced as well.
627632

628633
Coding styles
629634
=============
@@ -669,7 +674,8 @@ C++
669674

670675
### General rules
671676

672-
- Enforce language standard using corresponding compiler flag.
677+
- Enforce language standard using corresponding compiler flag. Don’t use
678+
standard extensions.
673679

674680
- Dependency on Boost is almost inevitable in large projects, don’t try to fight
675681
it by integrating newer compiler with the latest standard support. Don’t

0 commit comments

Comments
 (0)