Fix #255: suppress additional warnings: tabs, unused-imports

Closes #255.

Author: andreasabel

src/Main.hs

@@ -364,12 +364,25 @@ injectCode (Just (AlexPn _ ln _,code)) filename hdl = do
   hPutStrLn hdl code
 
 optsToInject :: Target -> [CLIFlags] -> [String]
-optsToInject target _ = optNoWarnings : "{-# LANGUAGE CPP #-}" : case target of
-  GhcTarget -> ["{-# LANGUAGE MagicHash #-}"]
-  _         -> []
+optsToInject target _ = concat
+  [ optNoWarnings
+  , [ "{-# LANGUAGE CPP #-}" ]
+  , [ "{-# LANGUAGE MagicHash #-}" | target == GhcTarget ]
+  ]
 
-optNoWarnings :: String
-optNoWarnings = "{-# OPTIONS_GHC -fno-warn-unused-binds -fno-warn-missing-signatures #-}"
+-- List here all harmless warnings caused by Alex-generated code.
+--
+-- These will be suppressed so that they are not printed
+-- when users turn on @-Wall@ in their lexer project.
+--
+optNoWarnings :: [String]
+optNoWarnings =
+  map (("{-# OPTIONS_GHC -fno-warn-" ++) . (++ " #-}"))
+    [ "missing-signatures"
+    , "tabs"
+    , "unused-binds"
+    , "unused-imports"
+    ]
 
 importsToInject :: Target -> [CLIFlags] -> String
 importsToInject _ cli = always_imports ++ debug_imports ++ glaexts_import

tests/Makefile

@@ -39,7 +39,7 @@ GHC_SHIPS_WITH_TEXT:=$(shell if [ $(GHC_MAJOR_VERSION) -gt 8 -o $(GHC_MAJOR_VERS
 # Turn off x-partial warning (new in GHC 9.8)
 WARNS_DEP_GHC_GTEQ_9_8:=$(shell if [ $(GHC_MAJOR_VERSION) -gt 9 -o $(GHC_MAJOR_VERSION) -ge 9 -a $(GHC_MINOR_VERSION) -ge 8 ]; then echo "-Wno-x-partial"; fi)
 
-HC_OPTS=-Wall $(WARNS_DEP_GHC_GTEQ_9_8) -fwarn-incomplete-uni-patterns -fno-warn-missing-signatures -fno-warn-unused-imports -fno-warn-tabs -Werror
+HC_OPTS=-Wall $(WARNS_DEP_GHC_GTEQ_9_8) -fwarn-incomplete-uni-patterns -Werror
 
 .PRECIOUS: %.n.hs %.g.hs %.o %.exe %.bin