Allow operators as record pattern synonym fields
authorMatthew Pickering <matthewtpickering@gmail.com>
Wed, 29 Mar 2017 20:09:08 +0000 (16:09 -0400)
committerBen Gamari <ben@smart-cactus.org>
Wed, 29 Mar 2017 20:44:43 +0000 (16:44 -0400)
Fixes #13454

Reviewers: austin, bgamari, dfeuer

Reviewed By: dfeuer

Subscribers: RyanGlScott, rwbarton, thomie

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

compiler/parser/Parser.y
testsuite/tests/patsyn/should_compile/T13454.hs [new file with mode: 0644]
testsuite/tests/patsyn/should_compile/all.T

index caa22dc..21f564e 100644 (file)
@@ -1362,8 +1362,8 @@ vars0 :: { [Located RdrName] }
         | varid vars0                 { $1 : $2 }
 
 cvars1 :: { [RecordPatSynField (Located RdrName)] }
-       : varid                        { [RecordPatSynField $1 $1] }
-       | varid ',' cvars1             {% addAnnotation (getLoc $1) AnnComma (getLoc $2) >>
+       : var                          { [RecordPatSynField $1 $1] }
+       | var ',' cvars1               {% addAnnotation (getLoc $1) AnnComma (getLoc $2) >>
                                          return ((RecordPatSynField $1 $1) : $3 )}
 
 where_decls :: { Located ([AddAnn]
diff --git a/testsuite/tests/patsyn/should_compile/T13454.hs b/testsuite/tests/patsyn/should_compile/T13454.hs
new file mode 100644 (file)
index 0000000..88a5441
--- /dev/null
@@ -0,0 +1,15 @@
+{-# LANGUAGE PatternSynonyms, ViewPatterns #-}
+module T13454 where
+
+pattern MkOp :: Op -> Exp -> Exp -> Exp
+pattern MkOp {(·), a, b} <- (splitOp -> Just ((·), a, b))
+  where MkOp (·) a b      = a · b
+
+data Exp = Val Int | Add Exp Exp | Mul Exp Exp deriving Show
+
+type Op = Exp -> Exp -> Exp
+
+splitOp :: Exp -> Maybe (Op, Exp, Exp)
+splitOp (Add a b) = Just (Add, a, b)
+splitOp (Mul a b) = Just (Mul, a, b)
+splitOp _         = Nothing
index 8fce7e9..fa8a3d8 100644 (file)
@@ -67,3 +67,4 @@ test('T13349b', normal, compile, [''])
 test('T13441', normal, compile, [''])
 test('T13441a', normal, compile, [''])
 test('T13441b', normal, compile_fail, [''])
+test('T13454', normal, compile, [''])