@@ -85,24 +85,18 @@ literals = mkPattern' match
8585 match (CppCharLiteral c) = return $ "'" <> T.singleton c <> "'"
8686 match (CppBooleanLiteral True) = return "true"
8787 match (CppBooleanLiteral False) = return "false"
88− match (CppArrayLiteral [x@CppArrayLiteral {}]) = -- Works around what appears to be a recent clang bug
89− mconcat <$> sequence
90− [ return $ runType arrayType
91− , return "{ { "
92− , prettyPrintCpp' x
93− , return " } }"
94− ]
88+ match (CppArrayLiteral []) = return $ runType arrayType <> "{}"
9589 match (CppArrayLiteral xs) = mconcat <$> sequence
9690 [ return $ runType arrayType
97− , return "{ "
91+ , return "{{ "
9892 , fmap (T.intercalate ", ") $ forM xs prettyPrintCpp'
99− , return " }"
93+ , return " }}"
10094 ]
10195 match (CppDataLiteral xs) = mconcat <$> sequence
10296 [ return $ runType (dataType $ length xs)
103− , return "{ "
97+ , return "{{ "
10498 , fmap (T.intercalate ", ") $ forM xs prettyPrintCpp'
105− , return " }"
99+ , return " }}"
106100 ]
107101 match (CppEnum name ty es) = mconcat <$> sequence
108102 [ return "enum"
@@ -276,15 +270,17 @@ literals = mkPattern' match
276270 , prettyPrintCpp' thens
277271 , maybe (return "") (fmap (" else " <>) . prettyPrintCpp') elses
278272 ]
279− match (CppSwitch cond cases@((case1,_):_:_)) = mconcat <$> sequence
273+ match (CppSwitch cond cases@((case1,_):_:_) default') = mconcat <$> sequence
280274 [ return "switch "
281275 , return . parensT $ prettyPrintCpp1 (cast case1 cond)
282276 , return " {\n"
283277 , withIndent $ do
284− cpps <- forM cases $ \(c, s) -> do
285− c' <- prettyPrintCpp' c
286− s' <- prettyPrintCpp' s
287− return $ "case " <> c' <> ": " <> s' <> ";"
278+ cpps <- forM (normalizeBlocks $
279+ cases ++ maybe [] (\c -> [(CppNoOp,c)]) default')
280+ $ \(c, s) -> do
281+ c' <- prettyPrintCpp' c
282+ s' <- prettyPrintCpp' s
283+ return $ label c <> c' <> ": " <> s' <> caseEnd s
288284 indentString <- currentIndent
289285 return $ T.intercalate "\n" $ map (indentString <>) cpps
290286 , return "\n"
@@ -297,6 +293,25 @@ literals = mkPattern' match
297293 cast CppBooleanLiteral {} cpp' = CppCast boolType cpp'
298294 cast CppCharLiteral {} cpp' = CppCast charType cpp'
299295 cast _ cpp' = cpp'
296+ normalizeBlocks :: [(Cpp,Cpp)] -> [(Cpp,Cpp)]
297+ normalizeBlocks cpps
298+ | any (isBlock . snd) cpps = addBlock <$> cpps
299+ | otherwise = cpps
300+ where
301+ addBlock (cpp1, cpp2@(CppBlock _)) = (cpp1, cpp2)
302+ addBlock (cpp1, cpp2) = (cpp1, CppBlock [cpp2])
303+ label :: Cpp -> Text
304+ label CppNoOp = "default"
305+ label _ = "case "
306+ caseEnd :: Cpp -> Text
307+ caseEnd (CppReturn {}) = ";"
308+ caseEnd (CppThrow {}) = ";"
309+ caseEnd (CppContinue) = ";"
310+ caseEnd (CppBlock stmts@(_:_))
311+ | (CppReturn {}) <- last stmts = ""
312+ | (CppThrow {}) <- last stmts = ""
313+ | CppContinue <- last stmts = ""
314+ caseEnd _ = " break;"
300315 match (CppReturn (CppBlock (cpp:cpps))) = mconcat <$> sequence
301316 [ do s <- prettyPrintCpp' cpp
302317 return $ T.dropWhile isSpace s
@@ -500,9 +515,6 @@ prettyPrintCpp' = A.runKleisli $ runPattern matchValue
500515 , [ binary Or " || " ]
501516 ]
502517
503−dotsTo :: Char -> Text -> Text
504−dotsTo chr' = T.map (\c -> if c == '.' then chr' else c)
505−
506518argstr :: (Text, Maybe CppType) -> Text
507519argstr (name, Nothing) = argStr name (Auto [])
508520argstr (name, Just typ)
@@ -538,6 +550,10 @@ isNoOp CppNoOp = True
538550isNoOp (CppComment [] CppNoOp) = True
539551isNoOp _ = False
540552
553+isBlock :: Cpp -> Bool
554+isBlock (CppBlock {}) = True
555+isBlock _ = False
556+
541557renderName :: Text -> Text
542558renderName name
543559 | Just ('*', name') <- T.uncons name = '$' `T.cons` name'