Unused import Data.Char in basic-bytestring lexer

[IgorErin]

I get the following warning when using basic-bytestring, In this code.

The qualified import of ‘Data.Char’ is redundant
      except perhaps to import instances from ‘Data.Char’
    To import instances alone, use: import Data.Char()
52 | import qualified Data.Char

As far as I understand, this is a generated import.
Is this how it should be, or am I doing something wrong?

[andreasabel]

The generated code is not entirely warning-free.
In the test-suite we suppress some warnings, one of them is about unused imports:

alex/tests/Makefile

Line 42 in 24d786c

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

Maybe making the code actually warning-free is too much effort. A proper solution would be to make a DSL for lexers in Haskell, generate DSL code, extract import from this code, and then translate this into actual haskell. However, the current implementation spits out Haskell directly and isn't suitable for introspection.

I suppose we could suppress these warnings with {-# OPTIONS_GHC -Wno-<WarningName> #-} pragmas put into the generated code.

[andreasabel]

I suppose fixing this minor issue does not warrant a release, does it?
The issue would then be fixed in the next regular release.