[project @ 2001-12-07 08:12:53 by sof]
[ghc-hetmet.git] / ghc / compiler / rename / RnNames.lhs
index 71bd508..078863f 100644 (file)
@@ -5,49 +5,43 @@
 
 \begin{code}
 module RnNames (
-       getGlobalNames
+       getGlobalNames, exportsFromAvail
     ) where
 
 #include "HsVersions.h"
 
-import CmdLineOpts    ( opt_NoImplicitPrelude, opt_WarnDuplicateExports, 
-                       opt_SourceUnchanged, opt_WarnUnusedBinds
-                     )
-
-import HsSyn   ( HsModule(..), HsDecl(..), TyClDecl(..),
-                 IE(..), ieName, 
-                 ForeignDecl(..), ForKind(..), isDynamicExtName,
-                 FixitySig(..), Sig(..), ImportDecl(..),
-                 collectTopBinders
-               )
-import RdrHsSyn        ( RdrNameIE, RdrNameImportDecl,
-                 RdrNameHsModule, RdrNameHsDecl
-               )
-import RnIfaces        ( getInterfaceExports, getDeclBinders, getDeclSysBinders,
-                 recordLocalSlurps, checkModUsage, findAndReadIface, outOfDate
-               )
+import CmdLineOpts     ( DynFlag(..) )
+
+import HsSyn           ( HsModule(..), HsDecl(..), IE(..), ieName, ImportDecl(..),
+                         ForeignDecl(..), 
+                         collectLocatedHsBinders
+                       )
+import RdrHsSyn                ( RdrNameIE, RdrNameImportDecl,
+                         RdrNameHsModule, RdrNameHsDecl
+                       )
+import RnIfaces                ( recordLocalSlurps )
+import RnHiFiles       ( getTyClDeclBinders, loadInterface )
 import RnEnv
 import RnMonad
 
 import FiniteMap
-import PrelInfo ( pRELUDE_Name, mAIN_Name, main_RDR )
-import UniqFM  ( lookupUFM )
-import Bag     ( bagToList )
-import Module  ( ModuleName, mkThisModule, pprModuleName, WhereFrom(..) )
+import PrelNames       ( pRELUDE_Name, mAIN_Name, main_RDR_Unqual, isUnboundName )
+import Module          ( ModuleName, moduleName, WhereFrom(..) )
+import Name            ( Name, nameSrcLoc, nameOccName )
 import NameSet
-import Name    ( Name, ExportFlag(..), ImportReason(..), Provenance(..),
-                 isLocallyDefined, setNameProvenance,
-                 nameOccName, getSrcLoc, pprProvenance, getNameProvenance,
-                 nameEnvElts
-               )
-import RdrName ( RdrName, rdrNameOcc, setRdrNameOcc, mkRdrQual, mkRdrUnqual, isQual, isUnqual )
-import OccName ( setOccNameSpace, dataName )
-import NameSet ( elemNameSet, emptyNameSet )
+import NameEnv
+import HscTypes                ( Provenance(..), ImportReason(..), GlobalRdrEnv,
+                         GenAvailInfo(..), AvailInfo, Avails, AvailEnv, 
+                         Deprecations(..), ModIface(..), emptyAvailEnv
+                       )
+import RdrName         ( rdrNameOcc, setRdrNameOcc )
+import OccName         ( setOccNameSpace, dataName )
+import NameSet         ( elemNameSet, emptyNameSet )
 import Outputable
-import Maybes  ( maybeToBool, catMaybes, mapMaybe )
-import UniqFM   ( emptyUFM, listToUFM, plusUFM_C )
-import Util    ( removeDups, equivClassesByUniq, sortLt )
-import List    ( partition )
+import Maybes          ( maybeToBool, catMaybes, mapMaybe )
+import ListSetOps      ( removeDups )
+import Util            ( sortLt )
+import List            ( partition )
 \end{code}
 
 
@@ -59,47 +53,31 @@ import List ( partition )
 %************************************************************************
 
 \begin{code}
-getGlobalNames :: RdrNameHsModule
-              -> RnMG (Maybe (GlobalRdrEnv,    -- Maps all in-scope things
-                              GlobalRdrEnv,    -- Maps just *local* things
-                              Avails,          -- The exported stuff
-                              AvailEnv,        -- Maps a name to its parent AvailInfo
-                                               -- Just for in-scope things only
-                              Maybe ParsedIface        -- The old interface file, if any
-                              ))
-                       -- Nothing => no need to recompile
-
-getGlobalNames (HsModule this_mod _ exports imports decls _ mod_loc)
-  =    -- These two fix-loops are to get the right
-       -- provenance information into a Name
-    fixRn ( \ ~(Just (rec_gbl_env, _, rec_export_avails, _, _)) ->
+getGlobalNames :: Module -> RdrNameHsModule
+              -> RnMG (GlobalRdrEnv,   -- Maps all in-scope things
+                       GlobalRdrEnv,   -- Maps just *local* things
+                       ExportAvails)   -- The exported stuff
 
-       let
-          rec_unqual_fn :: Name -> Bool        -- Is this chap in scope unqualified?
-          rec_unqual_fn = unQualInScope rec_gbl_env
-
-          rec_exp_fn :: Name -> ExportFlag
-          rec_exp_fn = mk_export_fn (availsToNameSet rec_export_avails)
-       in
-
-               -- PROCESS LOCAL DECLS
+getGlobalNames this_mod (HsModule _ _ _ imports decls _ mod_loc)
+  =            -- PROCESS LOCAL DECLS
                -- Do these *first* so that the correct provenance gets
                -- into the global name cache.
-       importsFromLocalDecls this_mod rec_exp_fn decls
-       `thenRn` \ (local_gbl_env, local_mod_avails) ->
+       importsFromLocalDecls this_mod decls            `thenRn` \ (local_gbl_env, local_mod_avails) ->
 
                -- PROCESS IMPORT DECLS
                -- Do the non {- SOURCE -} ones first, so that we get a helpful
                -- warning for {- SOURCE -} ones that are unnecessary
+       doptRn Opt_NoImplicitPrelude                            `thenRn` \ opt_no_prelude -> 
        let
+         all_imports        = mk_prel_imports opt_no_prelude ++ imports
          (source, ordinary) = partition is_source_import all_imports
          is_source_import (ImportDecl _ ImportByUserSource _ _ _ _) = True
          is_source_import other                                     = False
+
+         get_imports = importsFromImportDecl this_mod_name
        in
-       mapAndUnzipRn (importsFromImportDecl rec_unqual_fn) ordinary
-       `thenRn` \ (imp_gbl_envs1, imp_avails_s1) ->
-       mapAndUnzipRn (importsFromImportDecl rec_unqual_fn) source
-       `thenRn` \ (imp_gbl_envs2, imp_avails_s2) ->
+       mapAndUnzipRn get_imports ordinary      `thenRn` \ (imp_gbl_envs1, imp_avails_s1) ->
+       mapAndUnzipRn get_imports source        `thenRn` \ (imp_gbl_envs2, imp_avails_s2) ->
 
                -- COMBINE RESULTS
                -- We put the local env second, so that a local provenance
@@ -111,150 +89,105 @@ getGlobalNames (HsModule this_mod _ exports imports decls _ mod_loc)
 
            all_avails :: ExportAvails
            all_avails = foldr plusExportAvails local_mod_avails (imp_avails_s2 ++ imp_avails_s1)
-           (_, global_avail_env) = all_avails
        in
 
-               -- TRY FOR EARLY EXIT
-               -- We can't go for an early exit before this because we have to check
-               -- for name clashes.  Consider:
-               --
-               --      module A where          module B where
-               --         import B                h = True
-               --         f = h
-               --
-               -- Suppose I've compiled everything up, and then I add a
-               -- new definition to module B, that defines "f".
-               --
-               -- Then I must detect the name clash in A before going for an early
-               -- exit.  The early-exit code checks what's actually needed from B
-               -- to compile A, and of course that doesn't include B.f.  That's
-               -- why we wait till after the plusEnv stuff to do the early-exit.
-               
-       -- Check For eacly exit
-       checkErrsRn                             `thenRn` \ no_errs_so_far ->
-        if not no_errs_so_far then
-               -- Found errors already, so exit now
-               returnRn Nothing
-       else
-       checkEarlyExit this_mod                 `thenRn` \ (up_to_date, old_iface) ->
-       if up_to_date then
-               -- Interface files are sufficiently unchanged
-               putDocRn (text "Compilation IS NOT required")   `thenRn_`
-               returnRn Nothing
-       else
-       
-               -- RECORD BETTER PROVENANCES IN THE CACHE
-               -- The names in the envirnoment have better provenances (e.g. imported on line x)
-               -- than the names in the name cache.  We update the latter now, so that we
-               -- we start renaming declarations we'll get the good names
-               -- The isQual is because the qualified name is always in scope
-       updateProvenances (concat [names | (rdr_name, names) <- rdrEnvToList gbl_env, 
-                                          isQual rdr_name])    `thenRn_`
-       
-               -- PROCESS EXPORT LISTS
-       exportsFromAvail this_mod exports all_avails gbl_env    `thenRn` \ export_avails ->
-       
-       
                -- ALL DONE
-       returnRn (Just (gbl_env, local_gbl_env, export_avails, global_avail_env, old_iface))
-   )
+       returnRn (gbl_env, local_gbl_env, all_avails)
   where
-    all_imports = prel_imports ++ imports
+    this_mod_name = moduleName this_mod
 
        -- NB: opt_NoImplicitPrelude is slightly different to import Prelude ();
        -- because the former doesn't even look at Prelude.hi for instance declarations,
        -- whereas the latter does.
-    prel_imports | this_mod == pRELUDE_Name ||
-                  explicit_prelude_import ||
-                  opt_NoImplicitPrelude
-                = []
-
-                | otherwise = [ImportDecl pRELUDE_Name
-                                          ImportByUser
-                                          False        {- Not qualified -}
-                                          Nothing      {- No "as" -}
-                                          Nothing      {- No import list -}
-                                          mod_loc]
+    mk_prel_imports no_prelude
+       | this_mod_name == pRELUDE_Name ||
+         explicit_prelude_import ||
+         no_prelude
+       = []
+
+       | otherwise = [ImportDecl pRELUDE_Name
+                                 ImportByUser
+                                 False {- Not qualified -}
+                                 Nothing       {- No "as" -}
+                                 Nothing       {- No import list -}
+                                 mod_loc]
     
     explicit_prelude_import
       = not (null [ () | (ImportDecl mod _ _ _ _ _) <- imports, mod == pRELUDE_Name ])
 \end{code}
        
 \begin{code}
-checkEarlyExit mod_name
-  = traceRn (text "Considering whether compilation is required...")    `thenRn_`
-
-       -- Read the old interface file, if any, for the module being compiled
-    findAndReadIface doc_str mod_name False {- Not hi-boot -}  `thenRn` \ maybe_iface ->
-
-       -- CHECK WHETHER WE HAVE IT ALREADY
-    case maybe_iface of
-       Left err ->     -- Old interface file not found, so we'd better bail out
-                   traceRn (vcat [ptext SLIT("No old interface file for") <+> pprModuleName mod_name,
-                                  err])                        `thenRn_`
-                   returnRn (outOfDate, Nothing)
-
-       Right iface
-         | not opt_SourceUnchanged
-         ->    -- Source code changed
-            traceRn (nest 4 (text "source file changed or recompilation check turned off"))    `thenRn_` 
-            returnRn (False, Just iface)
-
-         | otherwise
-         ->    -- Source code unchanged and no errors yet... carry on 
-            checkModUsage (pi_usages iface)    `thenRn` \ up_to_date ->
-            returnRn (up_to_date, Just iface)
-  where
-       -- Only look in current directory, with suffix .hi
-    doc_str = sep [ptext SLIT("need usage info from"), pprModuleName mod_name]
-\end{code}
-       
-\begin{code}
-importsFromImportDecl :: (Name -> Bool)                -- OK to omit qualifier
+importsFromImportDecl :: ModuleName
                      -> RdrNameImportDecl
                      -> RnMG (GlobalRdrEnv, 
                               ExportAvails) 
 
-importsFromImportDecl is_unqual (ImportDecl imp_mod_name from qual_only as_mod import_spec iloc)
+importsFromImportDecl this_mod_name (ImportDecl imp_mod_name from qual_only as_mod import_spec iloc)
   = pushSrcLocRn iloc $
-    getInterfaceExports imp_mod_name from      `thenRn` \ (imp_mod, avails) ->
 
-    if null avails then
-       -- If there's an error in getInterfaceExports, (e.g. interface
+    loadInterface (ppr imp_mod_name <+> ptext SLIT("is directly imported"))
+                 imp_mod_name from                     `thenRn` \ iface ->
+    let
+       imp_mod          = mi_module iface
+       avails_by_module = mi_exports iface
+       deprecs          = mi_deprecs iface
+
+       avails :: Avails
+       avails = [ avail | (mod_name, avails) <- avails_by_module,
+                          mod_name /= this_mod_name,
+                          avail <- avails ]
+       -- If the module exports anything defined in this module, just ignore it.
+       -- Reason: otherwise it looks as if there are two local definition sites
+       -- for the thing, and an error gets reported.  Easiest thing is just to
+       -- filter them out up front. This situation only arises if a module
+       -- imports itself, or another module that imported it.  (Necessarily,
+       -- this invoves a loop.)  
+       --
+       -- Tiresome consequence: if you say
+       --      module A where
+       --         import B( AType )
+       --         type AType = ...
+       --
+       --      module B( AType ) where
+       --         import {-# SOURCE #-} A( AType )
+       --
+       -- then you'll get a 'B does not export AType' message.  Oh well.
+
+    in
+    if null avails_by_module then
+       -- If there's an error in loadInterface, (e.g. interface
        -- file not found) we get lots of spurious errors from 'filterImports'
        returnRn (emptyRdrEnv, mkEmptyExportAvails imp_mod_name)
     else
 
-    filterImports imp_mod_name import_spec avails   `thenRn` \ (filtered_avails, hides, explicits) ->
+       -- Complain if we import a deprecated module
+    ifOptRn Opt_WarnDeprecations       (
+       case deprecs of 
+         DeprecAll txt -> addWarnRn (moduleDeprec imp_mod_name txt)
+         other         -> returnRn ()
+    )                                                  `thenRn_`
 
-    qualifyImports imp_mod_name
-                  (not qual_only)      -- Maybe want unqualified names
-                  as_mod hides
-                  (improveAvails imp_mod iloc explicits 
-                                 is_unqual filtered_avails)
+       -- Filter the imports according to the import list
+    filterImports imp_mod_name from import_spec avails `thenRn` \ (filtered_avails, hides, explicits) ->
 
-
-improveAvails imp_mod iloc explicits is_unqual avails
-       -- We 'improve' the provenance by setting
-       --      (a) the import-reason field, so that the Name says how it came into scope
-       --              including whether it's explicitly imported
-       --      (b) the print-unqualified field
-  = map improve_avail avails
-  where
-    improve_avail (Avail n)      = Avail (improve n)
-    improve_avail (AvailTC n ns) = AvailTC (improve n) (map improve ns)
-
-    improve name = setNameProvenance name 
-                       (NonLocalDef (UserImport imp_mod iloc (is_explicit name)) 
-                                    (is_unqual name))
-    is_explicit name  = name `elemNameSet` explicits
+    let
+       unqual_imp = not qual_only              -- Maybe want unqualified names
+       qual_mod   = case as_mod of
+                       Nothing           -> imp_mod_name
+                       Just another_name -> another_name
+
+       mk_prov name = NonLocalDef (UserImport imp_mod iloc (name `elemNameSet` explicits)) 
+       gbl_env      = mkGlobalRdrEnv qual_mod unqual_imp mk_prov filtered_avails hides deprecs
+       exports      = mkExportAvails qual_mod unqual_imp gbl_env hides filtered_avails
+    in
+    returnRn (gbl_env, exports)
 \end{code}
 
 
 \begin{code}
-importsFromLocalDecls mod_name rec_exp_fn decls
-  = mapRn (getLocalDeclBinders mod rec_exp_fn) decls   `thenRn` \ avails_s ->
-
+importsFromLocalDecls this_mod decls
+  = mapRn (getLocalDeclBinders this_mod) decls `thenRn` \ avails_s ->
+       -- The avails that are returned don't include the "system" names
     let
        avails = concat avails_s
 
@@ -265,46 +198,63 @@ importsFromLocalDecls mod_name rec_exp_fn decls
        (_, dups) = removeDups compare all_names
     in
        -- Check for duplicate definitions
-    mapRn_ (addErrRn . dupDeclErr) dups                `thenRn_` 
-
-       -- Record that locally-defined things are available
-    recordLocalSlurps avails                   `thenRn_`
+       -- The complaint will come out as "Multiple declarations of Foo.f" because
+       -- since 'f' is in the env twice, the unQualInScope used by the error-msg
+       -- printer returns False.  It seems awkward to fix, unfortunately.
+    mapRn_ (addErrRn . dupDeclErr) dups                        `thenRn_` 
 
-       -- Build the environment
-    qualifyImports mod_name 
-                  True         -- Want unqualified names
-                  Nothing      -- no 'as M'
-                  []           -- Hide nothing
-                  avails
 
+       -- Record that locally-defined things are available
+    recordLocalSlurps (availsToNameSet avails)         `thenRn_`
+    let
+       mod_name   = moduleName this_mod
+       unqual_imp = True       -- Want unqualified names
+       mk_prov n  = LocalDef   -- Provenance is local
+       hides      = []         -- Hide nothing
+
+       gbl_env    = mkGlobalRdrEnv mod_name unqual_imp mk_prov avails hides NoDeprecs
+           -- NoDeprecs: don't complain about locally defined names
+           -- For a start, we may be exporting a deprecated thing
+           -- Also we may use a deprecated thing in the defn of another
+           -- deprecated things.  We may even use a deprecated thing in
+           -- the defn of a non-deprecated thing, when changing a module's 
+           -- interface
+
+       exports    = mkExportAvails mod_name unqual_imp gbl_env hides avails
+    in
+    returnRn (gbl_env, exports)
+
+---------------------------
+getLocalDeclBinders :: Module -> RdrNameHsDecl -> RnMG [AvailInfo]
+getLocalDeclBinders mod (TyClD tycl_decl)
+  =    -- For type and class decls, we generate Global names, with
+       -- no export indicator.  They need to be global because they get
+       -- permanently bound into the TyCons and Classes.  They don't need
+       -- an export indicator because they are all implicitly exported.
+    getTyClDeclBinders mod tycl_decl   `thenRn` \ (avail, sys_names) ->
+
+       -- Record that the system names are available
+    recordLocalSlurps (mkNameSet sys_names)    `thenRn_`
+    returnRn [avail]
+
+getLocalDeclBinders mod (ValD binds)
+  = mapRn new (collectLocatedHsBinders binds)          `thenRn` \ avails ->
+    returnRn avails
   where
-    mod = mkThisModule mod_name
-
-getLocalDeclBinders :: Module -> (Name -> ExportFlag)
-                   -> RdrNameHsDecl -> RnMG Avails
-getLocalDeclBinders mod rec_exp_fn (ValD binds)
-  = mapRn do_one (bagToList (collectTopBinders binds))
-  where
-    do_one (rdr_name, loc) = newLocalName mod rec_exp_fn rdr_name loc  `thenRn` \ name ->
-                            returnRn (Avail name)
-
-getLocalDeclBinders mod rec_exp_fn decl
-  = getDeclBinders (newLocalName mod rec_exp_fn) decl  `thenRn` \ maybe_avail ->
-    case maybe_avail of
-       Nothing    -> returnRn []               -- Instance decls and suchlike
-       Just avail -> returnRn [avail]
-
-newLocalName mod rec_exp_fn rdr_name loc 
-  = check_unqual rdr_name loc                  `thenRn_`
-    newTopBinder mod (rdrNameOcc rdr_name)     `thenRn` \ name ->
-    returnRn (setNameProvenance name (LocalDef loc (rec_exp_fn name)))
-  where
-       -- There should never be a qualified name in a binding position (except in instance decls)
-       -- The parser doesn't check this because the same parser parses instance decls
-    check_unqual rdr_name loc
-       | isUnqual rdr_name = returnRn ()
-       | otherwise         = qualNameErr (text "the binding for" <+> quotes (ppr rdr_name)) 
-                                         (rdr_name,loc)
+    new (rdr_name, loc) = newTopBinder mod rdr_name loc        `thenRn` \ name ->
+                         returnRn (Avail name)
+
+getLocalDeclBinders mod (ForD (ForeignImport nm _ _ loc))
+  = newTopBinder mod nm loc        `thenRn` \ name ->
+    returnRn [Avail name]
+getLocalDeclBinders mod (ForD _)
+  = returnRn []
+
+getLocalDeclBinders mod (FixD _)    = returnRn []
+getLocalDeclBinders mod (DeprecD _) = returnRn []
+getLocalDeclBinders mod (DefD _)    = returnRn []
+getLocalDeclBinders mod (InstD _)   = returnRn []
+getLocalDeclBinders mod (RuleD _)   = returnRn []
 \end{code}
 
 
@@ -319,22 +269,23 @@ available, and filters it through the import spec (if any).
 
 \begin{code}
 filterImports :: ModuleName                    -- The module being imported
+             -> WhereFrom                      -- Tells whether it's a {-# SOURCE #-} import
              -> Maybe (Bool, [RdrNameIE])      -- Import spec; True => hiding
              -> [AvailInfo]                    -- What's available
-             -> RnMG ([AvailInfo],             -- What's actually imported
-                      [AvailInfo],             -- What's to be hidden
-                                               -- (the unqualified version, that is)
-                       -- (We need to return both the above sets, because
-                       --  the qualified version is never hidden; so we can't
-                       --  implement hiding by reducing what's imported.)
+             -> RnMG ([AvailInfo],             -- "chosens"
+                      [AvailInfo],             -- "hides"
+                       -- The true imports are "chosens" - "hides"
+                       -- (It's convenient to return both the above sets, because
+                       --  the substraction can be done more efficiently when
+                       --  building the environment.)
                       NameSet)                 -- What was imported explicitly
 
        -- Complains if import spec mentions things that the module doesn't export
         -- Warns/informs if import spec contains duplicates.
-filterImports mod Nothing imports
+filterImports mod from Nothing imports
   = returnRn (imports, [], emptyNameSet)
 
-filterImports mod (Just (want_hiding, import_items)) avails
+filterImports mod from (Just (want_hiding, import_items)) total_avails
   = flatMapRn get_item import_items            `thenRn` \ avails_w_explicits ->
     let
        (item_avails, explicits_s) = unzip avails_w_explicits
@@ -343,22 +294,23 @@ filterImports mod (Just (want_hiding, import_items)) avails
     if want_hiding 
     then       
        -- All imported; item_avails to be hidden
-       returnRn (avails, item_avails, emptyNameSet)
+       returnRn (total_avails, item_avails, emptyNameSet)
     else
        -- Just item_avails imported; nothing to be hidden
        returnRn (item_avails, [], explicits)
   where
     import_fm :: FiniteMap OccName AvailInfo
     import_fm = listToFM [ (nameOccName name, avail) 
-                        | avail <- avails,
+                        | avail <- total_avails,
                           name  <- availNames avail]
        -- Even though availNames returns data constructors too,
        -- they won't make any difference because naked entities like T
        -- in an import list map to TcOccs, not VarOccs.
 
-    bale_out item = addErrRn (badImportItemErr mod item)       `thenRn_`
+    bale_out item = addErrRn (badImportItemErr mod from item)  `thenRn_`
                    returnRn []
 
+    get_item :: RdrNameIE -> RnMG [(AvailInfo, [Name])]
     get_item item@(IEModuleContents _) = bale_out item
 
     get_item item@(IEThingAll _)
@@ -367,14 +319,14 @@ filterImports mod (Just (want_hiding, import_items)) avails
          Just avail@(AvailTC _ [n]) ->         -- This occurs when you import T(..), but
                                                -- only export T abstractly.  The single [n]
                                                -- in the AvailTC is the type or class itself
-                                       addWarnRn (dodgyImportWarn mod item)    `thenRn_`
+                                       ifOptRn Opt_WarnMisc (addWarnRn (dodgyImportWarn mod item))     `thenRn_`
                                        returnRn [(avail, [availName avail])]
          Just avail                 -> returnRn [(avail, [availName avail])]
 
     get_item item@(IEThingAbs n)
       | want_hiding    -- hiding( C ) 
                        -- Here the 'C' can be a data constructor *or* a type/class
-      = case catMaybes [check_item item, check_item (IEThingAbs data_n)] of
+      = case catMaybes [check_item item, check_item (IEVar data_n)] of
                []     -> bale_out item
                avails -> returnRn [(a, []) | a <- avails]
                                -- The 'explicits' list is irrelevant when hiding
@@ -411,63 +363,12 @@ filterImports mod (Just (want_hiding, import_items)) avails
 %*                                                                     *
 %************************************************************************
 
-@qualifyImports@ takes the @ExportEnv@ after filtering through the import spec
-of an import decl, and deals with producing an @RnEnv@ with the 
-right qualified names.  It also turns the @Names@ in the @ExportEnv@ into
-fully fledged @Names@.
-
 \begin{code}
-qualifyImports :: ModuleName           -- Imported module
-              -> Bool                  -- True <=> want unqualified import
-              -> Maybe ModuleName      -- Optional "as M" part 
-              -> [AvailInfo]           -- What's to be hidden
-              -> Avails                -- Whats imported and how
-              -> RnMG (GlobalRdrEnv, ExportAvails)
-
-qualifyImports this_mod unqual_imp as_mod hides avails
-  = 
-       -- Make the name environment.  We're talking about a 
-       -- single module here, so there must be no name clashes.
-       -- In practice there only ever will be if it's the module
-       -- being compiled.
-    let
-       -- Add the things that are available
-       name_env1 = foldl add_avail emptyRdrEnv avails
-
-       -- Delete things that are hidden
-       name_env2 = foldl del_avail name_env1 hides
-
-       -- Create the export-availability info
-       export_avails = mkExportAvails qual_mod unqual_imp name_env2 avails
-    in
-    returnRn (name_env2, export_avails)
-
-  where
-    qual_mod = case as_mod of
-                 Nothing           -> this_mod
-                 Just another_name -> another_name
-
-    add_avail :: GlobalRdrEnv -> AvailInfo -> GlobalRdrEnv
-    add_avail env avail = foldl add_name env (availNames avail)
-
-    add_name env name
-       | unqual_imp = env2
-       | otherwise  = env1
-       where
-         env1 = addOneToGlobalRdrEnv env  (mkRdrQual qual_mod occ) name
-         env2 = addOneToGlobalRdrEnv env1 (mkRdrUnqual occ)        name
-         occ  = nameOccName name
-
-    del_avail env avail = foldl delOneFromGlobalRdrEnv env rdr_names
-                       where
-                         rdr_names = map (mkRdrUnqual . nameOccName) (availNames avail)
-
-
 mkEmptyExportAvails :: ModuleName -> ExportAvails
-mkEmptyExportAvails mod_name = (unitFM mod_name [], emptyUFM)
+mkEmptyExportAvails mod_name = (unitFM mod_name [], emptyNameEnv)
 
-mkExportAvails :: ModuleName -> Bool -> GlobalRdrEnv -> [AvailInfo] -> ExportAvails
-mkExportAvails mod_name unqual_imp name_env avails
+mkExportAvails :: ModuleName -> Bool -> GlobalRdrEnv -> [AvailInfo] -> [AvailInfo] -> ExportAvails
+mkExportAvails mod_name unqual_imp gbl_env hides avails 
   = (mod_avail_env, entity_avail_env)
   where
     mod_avail_env = unitFM mod_name unqual_avails 
@@ -482,16 +383,38 @@ mkExportAvails mod_name unqual_imp name_env avails
                  | otherwise      = mapMaybe prune avails
 
     prune (Avail n) | unqual_in_scope n = Just (Avail n)
-    prune (Avail n) | otherwise                = Nothing
+                    | otherwise                = Nothing
     prune (AvailTC n ns) | null uqs     = Nothing
                         | otherwise    = Just (AvailTC n uqs)
                         where
                           uqs = filter unqual_in_scope ns
 
-    unqual_in_scope n = unQualInScope name_env n
+    unqual_in_scope n = unQualInScope gbl_env n
+
 
-    entity_avail_env = listToUFM [ (name,avail) | avail <- avails, 
-                                                 name  <- availNames avail]
+    entity_avail_env  = mkNameEnv ([ (availName avail,avail) | avail <- effective_avails ]  ++
+                                       -- sigh - need to have the method/field names in
+                                       -- the environment also, so that export lists
+                                       -- can be computed precisely (cf. exportsFromAvail)
+                                  [ (name,avail) | avail <- effective_avails,
+                                                   name  <- avNames avail ] )
+
+    avNames (Avail n) = [n]
+    avNames (AvailTC n ns) = filter (/=n) ns
+
+       -- remove 'hides' names from the avail list.
+    effective_avails = foldl wipeOut avails hides
+      where
+        wipeOut as (Avail n)       = mapMaybe (delName n) as
+       wipeOut as (AvailTC n ns)  = foldl wipeOut as (map Avail ns)
+
+       delName x a@(Avail n) 
+         | n == x    = Nothing
+          | otherwise = Just a
+       delName x (AvailTC n ns) 
+         = case (filter (/=x) ns) of
+             [] -> Nothing
+             xs -> Just (AvailTC n xs)
 
 plusExportAvails ::  ExportAvails ->  ExportAvails ->  ExportAvails
 plusExportAvails (m1, e1) (m2, e2)
@@ -545,7 +468,7 @@ exportsFromAvail this_mod Nothing export_avails global_name_env
   = exportsFromAvail this_mod true_exports export_avails global_name_env
   where
     true_exports = Just $ if this_mod == mAIN_Name
-                          then [IEVar main_RDR]
+                          then [IEVar main_RDR_Unqual]
                                -- export Main.main *only* unless otherwise specified,
                           else [IEModuleContents this_mod]
                                -- but for all other modules export everything.
@@ -553,8 +476,10 @@ exportsFromAvail this_mod Nothing export_avails global_name_env
 exportsFromAvail this_mod (Just export_items) 
                 (mod_avail_env, entity_avail_env)
                 global_name_env
-  = foldlRn exports_from_item
-           ([], emptyFM, emptyAvailEnv) export_items   `thenRn` \ (_, _, export_avail_map) ->
+  = doptRn Opt_WarnDuplicateExports            `thenRn` \ warn_dup_exports ->
+    foldlRn (exports_from_item warn_dup_exports)
+           ([], emptyFM, emptyAvailEnv) export_items
+                                               `thenRn` \ (_, _, export_avail_map) ->
     let
        export_avails :: [AvailInfo]
        export_avails   = nameEnvElts export_avail_map
@@ -562,12 +487,11 @@ exportsFromAvail this_mod (Just export_items)
     returnRn export_avails
 
   where
-    exports_from_item :: ExportAccum -> RdrNameIE -> RnMG ExportAccum
+    exports_from_item :: Bool -> ExportAccum -> RdrNameIE -> RnMG ExportAccum
 
-    exports_from_item acc@(mods, occs, avails) ie@(IEModuleContents mod)
+    exports_from_item warn_dups acc@(mods, occs, avails) ie@(IEModuleContents mod)
        | mod `elem` mods       -- Duplicate export of M
-       = warnCheckRn opt_WarnDuplicateExports
-                     (dupModuleExport mod)     `thenRn_`
+       = warnCheckRn warn_dups (dupModuleExport mod)   `thenRn_`
          returnRn acc
 
        | otherwise
@@ -580,58 +504,51 @@ exportsFromAvail this_mod (Just export_items)
                                   in
                                   returnRn (mod:mods, occs', avails')
 
-    exports_from_item acc@(mods, occs, avails) ie
-       | not (maybeToBool maybe_in_scope) 
-       = failWithRn acc (unknownNameErr (ieName ie))
+    exports_from_item warn_dups acc@(mods, occs, avails) ie
+       = lookupSrcName global_name_env (ieName ie)     `thenRn` \ name -> 
 
-       | not (null dup_names)
-       = addNameClashErrRn rdr_name (name:dup_names)   `thenRn_`
-         returnRn acc
-
-#ifdef DEBUG
-       -- I can't see why this should ever happen; if the thing is in scope
-       -- at all it ought to have some availability
-       | not (maybeToBool maybe_avail)
-       = pprTrace "exportsFromAvail: curious Nothing:" (ppr name)
-         returnRn acc
-#endif
+               -- See what's available in the current environment
+         case lookupNameEnv entity_avail_env name of {
+           Nothing ->  -- Presumably this happens because lookupSrcName didn't find
+                       -- the name and returned an unboundName, which won't be in
+                       -- the entity_avail_env, of course
+                       WARN( not (isUnboundName name), ppr name )
+                       returnRn acc ;
 
-       | not enough_avail
-       = failWithRn acc (exportItemErr ie)
+           Just avail ->
 
-       | otherwise     -- Phew!  It's OK!  Now to check the occurrence stuff!
+               -- Filter out the bits we want
+         case filterAvail ie avail of {
+           Nothing ->  -- Not enough availability
+                          failWithRn acc (exportItemErr ie) ;
 
+           Just export_avail ->        
 
-       = warnCheckRn (ok_item ie avail) (dodgyExportWarn ie)   `thenRn_`
+               -- Phew!  It's OK!  Now to check the occurrence stuff!
+         warnCheckRn (ok_item ie avail) (dodgyExportWarn ie)   `thenRn_`
           check_occs ie occs export_avail                      `thenRn` \ occs' ->
          returnRn (mods, occs', addAvail avails export_avail)
+         }}
 
-       where
-         rdr_name        = ieName ie
-          maybe_in_scope  = lookupFM global_name_env rdr_name
-         Just (name:dup_names) = maybe_in_scope
-         maybe_avail        = lookupUFM entity_avail_env name
-         Just avail         = maybe_avail
-         maybe_export_avail = filterAvail ie avail
-         enough_avail       = maybeToBool maybe_export_avail
-         Just export_avail  = maybe_export_avail
-
-    ok_item (IEThingAll _) (AvailTC _ [n]) = False
-               -- This occurs when you import T(..), but
-               -- only export T abstractly.  The single [n]
-               -- in the AvailTC is the type or class itself
-    ok_item _ _ = True
+
+
+ok_item (IEThingAll _) (AvailTC _ [n]) = False
+  -- This occurs when you import T(..), but
+  -- only export T abstractly.  The single [n]
+  -- in the AvailTC is the type or class itself
+ok_item _ _ = True
 
 check_occs :: RdrNameIE -> ExportOccMap -> AvailInfo -> RnMG ExportOccMap
 check_occs ie occs avail 
-  = foldlRn check occs (availNames avail)
+  = doptRn Opt_WarnDuplicateExports    `thenRn` \ warn_dup_exports ->
+    foldlRn (check warn_dup_exports) occs (availNames avail)
   where
-    check occs name
+    check warn_dup occs name
       = case lookupFM occs name_occ of
          Nothing           -> returnRn (addToFM occs name_occ (name, ie))
          Just (name', ie') 
            | name == name' ->  -- Duplicate export
-                               warnCheckRn opt_WarnDuplicateExports
+                               warnCheckRn warn_dup
                                            (dupExportWarn name_occ ie ie')
                                `thenRn_` returnRn occs
 
@@ -639,12 +556,6 @@ check_occs ie occs avail
                                failWithRn occs (exportClashErr name_occ ie ie')
       where
        name_occ = nameOccName name
-       
-mk_export_fn :: NameSet -> (Name -> ExportFlag)
-mk_export_fn exported_names
-  = \name -> if name `elemNameSet` exported_names
-            then Exported
-            else NotExported
 \end{code}
 
 %************************************************************************
@@ -654,9 +565,13 @@ mk_export_fn exported_names
 %************************************************************************
 
 \begin{code}
-badImportItemErr mod ie
-  = sep [ptext SLIT("Module"), quotes (pprModuleName mod), 
+badImportItemErr mod from ie
+  = sep [ptext SLIT("Module"), quotes (ppr mod), source_import,
         ptext SLIT("does not export"), quotes (ppr ie)]
+  where
+    source_import = case from of
+                     ImportByUserSource -> ptext SLIT("(hi-boot interface)")
+                     other              -> empty
 
 dodgyImportWarn mod item = dodgyMsg (ptext SLIT("import")) item
 dodgyExportWarn     item = dodgyMsg (ptext SLIT("export")) item
@@ -667,7 +582,7 @@ dodgyMsg kind item@(IEThingAll tc)
          ptext SLIT("but it has none; it is a type synonym or abstract type or class") ]
          
 modExportErr mod
-  = hsep [ ptext SLIT("Unknown module in export list: module"), quotes (pprModuleName mod)]
+  = hsep [ ptext SLIT("Unknown module in export list: module"), quotes (ppr mod)]
 
 exportItemErr export_item
   = sep [ ptext SLIT("The export item") <+> quotes (ppr export_item),
@@ -680,13 +595,10 @@ exportClashErr occ_name ie1 ie2
 
 dupDeclErr (n:ns)
   = vcat [ptext SLIT("Multiple declarations of") <+> quotes (ppr n),
-         nest 4 (vcat (map pp sorted_ns))]
+         nest 4 (vcat (map ppr sorted_locs))]
   where
-    sorted_ns = sortLt occ'ed_before (n:ns)
-
-    occ'ed_before a b = LT == compare (getSrcLoc a) (getSrcLoc b)
-
-    pp n      = pprProvenance (getNameProvenance n)
+    sorted_locs = sortLt occ'ed_before (map nameSrcLoc (n:ns))
+    occ'ed_before a b = LT == compare a b
 
 dupExportWarn occ_name ie1 ie2
   = hsep [quotes (ppr occ_name), 
@@ -695,6 +607,10 @@ dupExportWarn occ_name ie1 ie2
 
 dupModuleExport mod
   = hsep [ptext SLIT("Duplicate"),
-         quotes (ptext SLIT("Module") <+> pprModuleName mod), 
+         quotes (ptext SLIT("Module") <+> ppr mod), 
           ptext SLIT("in export list")]
+
+moduleDeprec mod txt
+  = sep [ ptext SLIT("Module") <+> quotes (ppr mod) <+> ptext SLIT("is deprecated:"), 
+         nest 4 (ppr txt) ]      
 \end{code}