Skip to content

Commit

Permalink
Fix call and parameter pretty printing
Browse files Browse the repository at this point in the history
Call was missing the call keyword from pretty printing.
And while StParameter uses the same decl as everywhere else, even in
Fortran77 it uses `=` for assigning values rather than the old style
initial value syntax.
  • Loading branch information
Raoul Hidalgo Charman authored and raehik committed Jul 28, 2023
1 parent d149c8a commit 239f7b1
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions src/Language/Fortran/PrettyPrint.hs
Original file line number Diff line number Diff line change
Expand Up @@ -542,7 +542,9 @@ instance Pretty (Statement a) where
| v >= Fortran90 = "namelist" <+> pprint' v namelist
| otherwise = tooOld v "Namelist statement" Fortran90

pprint' v (StParameter _ _ aDecls) = "parameter" <+> parens (pprint' v aDecls)
-- We reuse the declaration node, but parameter statements use `=` even in
-- the older standards
pprint' _ (StParameter _ _ aDecls) = "parameter" <+> parens (pprint' Fortran90 aDecls)

pprint' v (StExternal _ _ vars) = "external" <+> pprint' v vars
pprint' v (StIntrinsic _ _ vars) = "intrinsic" <+> pprint' v vars
Expand Down Expand Up @@ -650,7 +652,7 @@ instance Pretty (Statement a) where
pprint' v (StGotoComputed _ _ labels target) =
"goto" <+> parens (pprint' v labels) <+> pprint' v target

pprint' v (StCall _ _ name args) = pprint' v name <+> parens (pprint' v args)
pprint' v (StCall _ _ name args) = "call" <+> pprint' v name <+> parens (pprint' v args)

pprint' _ (StContinue _ _) = "continue"

Expand Down

0 comments on commit 239f7b1

Please sign in to comment.