Tidy up the parsing a bit
authorLuke Lau <luke_lau@icloud.com>
Wed, 5 Jun 2019 21:12:52 +0000 (22:12 +0100)
committerLuke Lau <luke_lau@icloud.com>
Fri, 8 Nov 2019 15:07:39 +0000 (15:07 +0000)
The Text.ParserCombinators.ReadPrec/Text.ParserCominators.ReadP split
adds a bit of lifting cruft, so lets try to sweep that away.

AST.hs

diff --git a/AST.hs b/AST.hs
index 1e505af4f47116e401e2523f10c90ac1c466c360..ae18a52347ce91c37336a06f891bae690ba691ec 100644 (file)
--- a/AST.hs
+++ b/AST.hs
@@ -30,10 +30,7 @@ instance Read Expr where
           parseVar = Var <$> lift (munch1 isAlpha)
           parseBinOp s prc op = prec prc $ do
             a <- step readPrec
-            lift $ do
-              skipSpaces
-              string s
-              skipSpaces
+            spaced $ string s
             b <- readPrec
             return (BinOp op a b)
           parseCall = do
@@ -43,13 +40,14 @@ instance Read Expr where
                               (skipSpaces >> char ',' >> skipSpaces)
             return (Call func params)
           parseIf = do
-            lift $ skipSpaces >> string "if" >> skipSpaces
+            spaced $ string "if" 
             cond <- readPrec
-            lift $ skipSpaces >> string "then" >> skipSpaces
+            spaced $ string "then"
             thenE <- readPrec
-            lift $ skipSpaces >> string "else" >> skipSpaces
+            spaced $ string "else"
             elseE <- readPrec
             return (If cond thenE elseE)
+          spaced f = lift $ skipSpaces >> f >> skipSpaces
             
 data Prototype = Prototype String [String]
   deriving Show