From d600bae7360f483f0a12f9cfde78a258d42d87de Mon Sep 17 00:00:00 2001
From: Marco Benelli <marco.benelli@qt.io>
Date: Wed, 6 Sep 2017 11:55:48 +0200
Subject: [PATCH] QmlJs: better indentation for switch common cases

Do not add an empty line between common cases in a `switch` block,
according the best practices for javascript style.

Task-number: QTCREATORBUG-18515
Change-Id: I8b665eabb9b1d5b617d608e430e8b9137dc32032
Reviewed-by: Thomas Hartmann <thomas.hartmann@qt.io>
---
 src/libs/qmljs/qmljsreformatter.cpp | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/libs/qmljs/qmljsreformatter.cpp b/src/libs/qmljs/qmljsreformatter.cpp
index a3d8481d58..c78f52f5b8 100644
--- a/src/libs/qmljs/qmljsreformatter.cpp
+++ b/src/libs/qmljs/qmljsreformatter.cpp
@@ -1102,7 +1102,8 @@ protected:
         out("case ", ast->caseToken);
         accept(ast->expression);
         out(ast->colonToken);
-        lnAcceptIndented(ast->statements);
+        if (ast->statements)
+            lnAcceptIndented(ast->statements);
         return false;
     }
 
-- 
GitLab