Skip to content

Commit 7817c59

Browse files
MarcoFalkeRandyMcMillan
authored andcommitted
doc: Remove dev note section on includes
1 parent 70afca6 commit 7817c59

File tree

2 files changed

+4
-8
lines changed

2 files changed

+4
-8
lines changed

doc/developer-notes.md

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1159,14 +1159,6 @@ namespace {
11591159

11601160
- *Rationale*: Avoids confusion about the namespace context.
11611161

1162-
- Use `#include <primitives/transaction.h>` bracket syntax instead of
1163-
`#include "primitives/transactions.h"` quote syntax.
1164-
1165-
- *Rationale*: Bracket syntax is less ambiguous because the preprocessor
1166-
searches a fixed list of include directories without taking location of the
1167-
source file into account. This allows quoted includes to stand out more when
1168-
the location of the source file actually is relevant.
1169-
11701162
- Use include guards to avoid the problem of double inclusion. The header file
11711163
`foo/bar.h` should use the include guard identifier `BITCOIN_FOO_BAR_H`, e.g.
11721164

test/lint/lint-includes.py

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -166,6 +166,10 @@ def main():
166166

167167
# Enforce bracket syntax includes
168168
quote_syntax_inclusions = find_quote_syntax_inclusions()
169+
# *Rationale*: Bracket syntax is less ambiguous because the preprocessor
170+
# searches a fixed list of include directories without taking location of the
171+
# source file into account. This allows quoted includes to stand out more when
172+
# the location of the source file actually is relevant.
169173

170174
if quote_syntax_inclusions:
171175
print("Please use bracket syntax includes (\"#include <foo.h>\") instead of quote syntax includes:")

0 commit comments

Comments
 (0)