Skip to content

Commit

Permalink
suppress linter errors due to not including the directory when naming .h
Browse files Browse the repository at this point in the history
files
  • Loading branch information
jeffdonahue authored and shelhamer committed Feb 26, 2014
1 parent ba95a87 commit dd7854c
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions scripts/cpplint.py
Original file line number Diff line number Diff line change
Expand Up @@ -148,6 +148,7 @@
'build/header_guard',
'build/include',
'build/include_alpha',
'build/include_dir',
'build/include_order',
'build/include_what_you_use',
'build/namespaces',
Expand Down Expand Up @@ -207,7 +208,7 @@
# flag. By default all errors are on, so only add here categories that should be
# off by default (i.e., categories that must be enabled by the --filter= flags).
# All entries here should start with a '-' or '+', as in the --filter= flag.
_DEFAULT_FILTERS = ['-build/include_alpha']
_DEFAULT_FILTERS = ['-build/include_alpha', '-build/include_dir']

# We used to check for high-bit characters, but after much discussion we
# decided those were OK, as long as they were in UTF-8 and didn't represent
Expand Down Expand Up @@ -3585,7 +3586,7 @@ def CheckIncludeLine(filename, clean_lines, linenum, include_state, error):

# "include" should use the new style "foo/bar.h" instead of just "bar.h"
if _RE_PATTERN_INCLUDE_NEW_STYLE.search(line):
error(filename, linenum, 'build/include', 4,
error(filename, linenum, 'build/include_dir', 4,
'Include the directory when naming .h files')

# we shouldn't include a file more than once. actually, there are a
Expand Down

0 comments on commit dd7854c

Please sign in to comment.