From: Roberto Raggi Date: Fri, 6 May 2011 08:08:27 +0000 (+0200) Subject: Don't visit the sub tree of an AST::UiScriptBinding node. X-Git-Tag: upstream/5.2.1~4394^2~151 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=56d4f0cc4552e11bee24cd977ada365df645889a;p=platform%2Fupstream%2Fqtdeclarative.git Don't visit the sub tree of an AST::UiScriptBinding node. ProcessAST needs to look only at QML nodes and as we know it is impossible to find QML nodes inside JavaScript statements. --- diff --git a/src/declarative/qml/qdeclarativescriptparser.cpp b/src/declarative/qml/qdeclarativescriptparser.cpp index 352e341..e9be923 100644 --- a/src/declarative/qml/qdeclarativescriptparser.cpp +++ b/src/declarative/qml/qdeclarativescriptparser.cpp @@ -724,7 +724,7 @@ bool ProcessAST::visit(AST::UiScriptBinding *node) while (propertyCount--) _stateStack.pop(); - return true; + return false; } static QList collectCommas(AST::UiArrayMemberList *members) @@ -759,7 +759,7 @@ bool ProcessAST::visit(AST::UiArrayBinding *node) error.setLine(this->location(propertyName).start.line); error.setColumn(this->location(propertyName).start.column); _parser->_errors << error; - return 0; + return false; } accept(node->members);