Unquote ‘import’ in bad import error message
authorPhil Ruffwind <rf@rufflewind.com>
Tue, 17 Jan 2017 20:55:28 +0000 (15:55 -0500)
committerBen Gamari <ben@smart-cactus.org>
Tue, 17 Jan 2017 20:56:28 +0000 (15:56 -0500)
    In module ‘Prelude’:
      ‘True’ is a data constructor of ‘Bool’
    To import it use
      ‘import’ Prelude( Bool( True ) )

The quotes around `import` don't make any sense.

Test Plan: manual

Reviewers: austin, mpickering, bgamari

Reviewed By: mpickering, bgamari

Subscribers: dfeuer, thomie

Differential Revision: https://phabricator.haskell.org/D2935

compiler/rename/RnNames.hs
testsuite/tests/module/mod90.stderr
testsuite/tests/rename/should_fail/T10668.stderr
testsuite/tests/rename/should_fail/T5385.stderr

index 8a7529d..9d2de74 100644 (file)
@@ -1488,11 +1488,11 @@ badImportItemErrDataCon dataType_occ iface decl_spec ie
              <+> text "is a data constructor of"
              <+> quotes dataType
          , text "To import it use"
-         , nest 2 $ quotes (text "import")
+         , nest 2 $ text "import"
              <+> ppr (is_mod decl_spec)
              <> parens_sp (dataType <> parens_sp datacon)
          , text "or"
-         , nest 2 $ quotes (text "import")
+         , nest 2 $ text "import"
              <+> ppr (is_mod decl_spec)
              <> parens_sp (dataType <> text "(..)")
          ]
index 9febfe3..78a0040 100644 (file)
@@ -1,8 +1,8 @@
 
-mod90.hs:3:16:
+mod90.hs:3:16: error:
     In module ‘Prelude’:
       ‘Left’ is a data constructor of ‘Either’
     To import it use
-      ‘import’ Prelude( Either( Left ) )
+      import Prelude( Either( Left ) )
     or
-      ‘import’ Prelude( Either(..) )
+      import Prelude( Either(..) )
index 8c96fad..4e602b2 100644 (file)
@@ -3,6 +3,6 @@ T10668.hs:3:27: error:
     In module ‘Data.Type.Equality’:
       ‘Refl’ is a data constructor of ‘(:~:)’
     To import it use
-      ‘import’ Data.Type.Equality( (:~:)( Refl ) )
+      import Data.Type.Equality( (:~:)( Refl ) )
     or
-      ‘import’ Data.Type.Equality( (:~:)(..) )
+      import Data.Type.Equality( (:~:)(..) )
index 677c31f..dd825c6 100644 (file)
@@ -1,8 +1,8 @@
 
-T5385.hs:3:16:
+T5385.hs:3:16: error:
     In module ‘T5385a’:
       ‘(:::)’ is a data constructor of ‘T’
     To import it use
-      ‘import’ T5385a( T( (:::) ) )
+      import T5385a( T( (:::) ) )
     or
-      ‘import’ T5385a( T(..) )
+      import T5385a( T(..) )