QmlDesigner: Fix tests after parser update

Since the last parser update the source location is valid
even if there is no semicolon. The length is 0 in this case, though.

Change-Id: Ie2ff425b54ef39d9db66119236d5e8f56677b748
Reviewed-by: Vikas Pachdha <vikas.pachdha@qt.io>

1 files changed, 5 insertions, 3 deletions

diff --git a/src/plugins/qmldesigner/designercore/filemanager/addpropertyvisitor.cpp b/src/plugins/qmldesigner/designercore/filemanager/addpropertyvisitor.cpp
index 3c04d341bbd..47a7af5fc3b 100644
--- a/src/plugins/qmldesigner/designercore/filemanager/addpropertyvisitor.cpp
+++ b/src/plugins/qmldesigner/designercore/filemanager/addpropertyvisitor.cpp
@@ -99,10 +99,12 @@ void AddPropertyVisitor::addInMembers(QmlJS::AST::UiObjectInitializer *initializ
 
         // Find out if the previous members ends with semicolon.
         if (auto member = QmlJS::AST::cast<QmlJS::AST::UiScriptBinding*>(insertAfter->member)) {
-            if (auto stmt = QmlJS::AST::cast<QmlJS::AST::ExpressionStatement*>(member->statement))
-                previousMemberSemicolon = stmt->semicolonToken.isValid();
-            else
+            if (auto stmt = QmlJS::AST::cast<QmlJS::AST::ExpressionStatement *>(member->statement)) {
+                previousMemberSemicolon = stmt->semicolonToken.isValid()
+                                          && stmt->semicolonToken.length > 0;
+            } else {
                 previousMemberSemicolon = endOfPreviousMember.isValid();
+            }
         } else {
             previousMemberSemicolon = endOfPreviousMember.isValid();
         }