[project @ 2000-07-06 16:31:45 by simonpj]
authorsimonpj <unknown>
Thu, 6 Jul 2000 16:31:46 +0000 (16:31 +0000)
committersimonpj <unknown>
Thu, 6 Jul 2000 16:31:46 +0000 (16:31 +0000)
commit525898a970c625753c33490318762c2b4c2770a9
treef6c0ab7b0edde78fa8d4b0a03b60c02a051086ba
parent89cefac83e7e468ddc0fdc62b3f4e076e97a7d51
[project @ 2000-07-06 16:31:45 by simonpj]
* Improve the warning "M is imported but nothing from it is used"
  In particular, don't warn if some instances from it are imported.

  It's pretty much impossible to do the Right Thing always.
  A comment in Rename.lhs says
-- NOTE: Consider
--       module This
-- import M ()
--
--  The import M() is not *necessarily* redundant, even if
--   we suck in no instance decls from M (e.g. it contains
--  no instance decls, or This contains no code).  It may be
--  that we import M solely to ensure that M's orphan instance
--  decls (or those in its imports) are visible to people who
--  import This.  Sigh.
--  There's really no good way to detect this, so the error message
--  in RnEnv.warnUnusedModules is weakened instead

* Minor comment changes to RnIfaces.lhs

* Use NameEnv instead of UFM in TcEnv (tidy up only)
ghc/compiler/rename/Rename.lhs
ghc/compiler/rename/RnEnv.lhs
ghc/compiler/rename/RnIfaces.lhs
ghc/compiler/rename/RnMonad.lhs
ghc/compiler/typecheck/TcEnv.lhs