Skip to content

Commit

Permalink
Need to add hpp headers to valid extension when they are and aren't p…
Browse files Browse the repository at this point in the history
…rovided via command line
  • Loading branch information
lucasw committed Sep 21, 2021
1 parent be2160f commit 73013e6
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions src/roslint/cpplint.py
Original file line number Diff line number Diff line change
Expand Up @@ -575,8 +575,6 @@ def ProcessHppHeadersOption(val):
global _hpp_headers
try:
_hpp_headers = set(val.split(','))
# Automatically append to extensions list so it does not have to be set 2 times
_valid_extensions.update(_hpp_headers)
except ValueError:
PrintUsage('Header extensions must be comma separated list.')

Expand Down Expand Up @@ -6028,6 +6026,9 @@ def ProcessConfigOverrides(filename):
for filter in reversed(cfg_filters):
_AddFilters(filter)

# These are either the default headers or were modified above by options
_valid_extensions.update(_hpp_headers)

return True


Expand Down

0 comments on commit 73013e6

Please sign in to comment.