Skip to content

Commit

Permalink
activate next -Wall warning to squash
Browse files Browse the repository at this point in the history
See #347.
  • Loading branch information
garfieldnate committed Oct 3, 2024
1 parent 062c26b commit fccaa0d
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions SConstruct
Original file line number Diff line number Diff line change
Expand Up @@ -232,7 +232,7 @@ lnflags = []
libs = ['Soar']

# TODO: Enabling all the warnings is a WIP! These are very thorough and need to be disabled for
# parts we don't control, like the SWIG-generated code.
# parts we don't control, like the SWIG-generated code. See https://github.com/SoarGroup/Soar/issues/347.
if compiler == "msvc":
pass
# show all warnings
Expand All @@ -246,7 +246,7 @@ else:
# cflags.extend(['-Werror'])

# We're starting with something simple. We'll add more as we go.
cflags.extend(['-Wunused-variable', '-Wreorder'])
cflags.extend(['-Wunused-variable', '-Wreorder', '-Wunused-but-set-variable'])

# warning doesn't exist in Apple's clang
if sys.platform != 'darwin':
Expand Down

0 comments on commit fccaa0d

Please sign in to comment.