diff --git a/share/qtcreator/templates/qt4project/widget.ui b/share/qtcreator/templates/qt4project/widget.ui index f4e94939e46edcd6e9cecf706a21a2947c3de60a..a804f2044eb4fa69dd5833134fcf3a29e9f5cba3 100644 --- a/share/qtcreator/templates/qt4project/widget.ui +++ b/share/qtcreator/templates/qt4project/widget.ui @@ -1,5 +1,5 @@ <ui version="4.0"> - <class>%CLASS%Class</class> + <class>%CLASS%</class> <widget class="%BASECLASS%" name="%CLASS%" > <property name="geometry" > <rect> diff --git a/src/plugins/cpptools/cppcodecompletion.cpp b/src/plugins/cpptools/cppcodecompletion.cpp index 8805d4555dca6c56a47753d6558d2767e5683eef..6157204867fb29585d7214ce28d8a9daef33229d 100644 --- a/src/plugins/cpptools/cppcodecompletion.cpp +++ b/src/plugins/cpptools/cppcodecompletion.cpp @@ -1128,7 +1128,7 @@ void CppCodeCompletion::complete(const TextEditor::CompletionItem &item) if (function->argumentCount() == 0) { extraChars += QLatin1Char('<'); } - } else { + } else if (! function->isAmbiguous()) { extraChars += QLatin1Char('('); // If the function takes no arguments, automatically place the closing parenthesis diff --git a/src/plugins/cpptools/cppmodelmanager.cpp b/src/plugins/cpptools/cppmodelmanager.cpp index 5eda4b77e54ff171c2f081bc3b6f916807352a1a..43bc047f35d53c15d00db55713baf3c664b0c7e2 100644 --- a/src/plugins/cpptools/cppmodelmanager.cpp +++ b/src/plugins/cpptools/cppmodelmanager.cpp @@ -214,7 +214,6 @@ private: Document::Ptr m_currentDoc; QSet<QString> m_todo; QSet<QString> m_processed; - QFutureSynchronizer<void> m_synchronizer; }; } // namespace Internal @@ -224,9 +223,7 @@ CppPreprocessor::CppPreprocessor(QPointer<CppModelManager> modelManager) : snapshot(modelManager->snapshot()), m_modelManager(modelManager), preprocess(this, &env) -{ - m_synchronizer.setCancelOnWait(true); -} +{ } CppPreprocessor::~CppPreprocessor() { } @@ -510,7 +507,9 @@ void CppPreprocessor::sourceNeeded(QString &fileName, IncludeType type, snapshot.insert(doc->fileName(), doc); m_todo.remove(fileName); - m_synchronizer.addFuture(QtConcurrent::run(Process(m_modelManager), doc)); + Process process(m_modelManager); + + process(doc); (void) switchDocument(previousDoc); } diff --git a/src/plugins/debugger/debuggermanager.cpp b/src/plugins/debugger/debuggermanager.cpp index 7c1b583f6262e4c3891f0883f7b037427a0d7348..eb14caf254f8a9b9fb5f10a71b554a9fae769e4e 100644 --- a/src/plugins/debugger/debuggermanager.cpp +++ b/src/plugins/debugger/debuggermanager.cpp @@ -1195,10 +1195,11 @@ static bool isAllowedTransition(int from, int to) { return (from == -1) || (from == DebuggerProcessNotReady && to == DebuggerProcessStartingUp) - || (from == DebuggerProcessStartingUp && to == DebuggerInferiorStopped) + //|| (from == DebuggerProcessStartingUp && to == DebuggerInferiorStopped) || (from == DebuggerInferiorStopped && to == DebuggerInferiorRunningRequested) || (from == DebuggerInferiorRunningRequested && to == DebuggerInferiorRunning) || (from == DebuggerInferiorRunning && to == DebuggerInferiorStopRequested) + || (from == DebuggerInferiorRunning && to == DebuggerInferiorStopped) || (from == DebuggerInferiorStopRequested && to == DebuggerInferiorStopped) || (to == DebuggerProcessNotReady); } @@ -1214,7 +1215,6 @@ void DebuggerManager::setStatus(int status) if (!isAllowedTransition(m_status, status)) qDebug() << "UNEXPECTED TRANSITION: " << m_status << status; - m_status = status; const bool started = status == DebuggerInferiorRunning diff --git a/src/plugins/designer/cpp/formclasswizardparameters.cpp b/src/plugins/designer/cpp/formclasswizardparameters.cpp index 2c2cde09cf3ad4ced2bc5a3b75808fee2a1ec23c..c7f7407e92588ed568cfa9675ebcfc122cf32968 100644 --- a/src/plugins/designer/cpp/formclasswizardparameters.cpp +++ b/src/plugins/designer/cpp/formclasswizardparameters.cpp @@ -162,6 +162,7 @@ bool FormClassWizardParameters::generateCpp(QString *header, QString *source, in if (languageChange) { sourceStr << '\n' << namespaceIndent << "void " << unqualifiedClassName << "::" << "changeEvent(QEvent *e)\n" << namespaceIndent << "{\n" + << namespaceIndent << indent << formBaseClass << "::changeEvent(e);\n" << namespaceIndent << indent << "switch (e->type()) {\n" << namespaceIndent << indent << "case QEvent::LanguageChange:\n" << namespaceIndent << indent << indent; if (embedding != InheritedUiClass) diff --git a/src/plugins/git/gitclient.cpp b/src/plugins/git/gitclient.cpp index 8baabe03a2afc0de8d46ada86202dedff17735c2..c7558f1a18f9ce111512e6f0b10e8a3e76ea4449 100644 --- a/src/plugins/git/gitclient.cpp +++ b/src/plugins/git/gitclient.cpp @@ -235,9 +235,9 @@ void GitClient::diff(const QString &workingDirectory, if (Git::Constants::debug) qDebug() << "diff" << workingDirectory << fileName; QStringList arguments; - arguments << QLatin1String("diff"); + arguments << QLatin1String("diff") << QLatin1String(noColorOption); if (!fileName.isEmpty()) - arguments << diffArgs << QLatin1String(noColorOption) << QLatin1String("--") << fileName; + arguments << diffArgs << QLatin1String("--") << fileName; const QString kind = QLatin1String(Git::Constants::GIT_DIFF_EDITOR_KIND); const QString title = tr("Git Diff %1").arg(fileName); diff --git a/src/shared/cplusplus/AST.cpp b/src/shared/cplusplus/AST.cpp index 19373cdadae9678ae11f8e1bf2399b48102f3ba0..1f051146bf497c85167c933b7cc27738f44d7aaa 100644 --- a/src/shared/cplusplus/AST.cpp +++ b/src/shared/cplusplus/AST.cpp @@ -1682,6 +1682,10 @@ FunctionDeclaratorAST *FunctionDeclaratorAST::clone(MemoryPool *pool) const void FunctionDeclaratorAST::accept0(ASTVisitor *visitor) { if (visitor->visit(this)) { + accept(parameters, visitor); + for (SpecifierAST *it = cv_qualifier_seq; it; it = it->next) + accept(it, visitor); + accept(exception_specification, visitor); } visitor->endVisit(this); } diff --git a/src/shared/cplusplus/AST.h b/src/shared/cplusplus/AST.h index 9093cb650f49ea9cdd03ab5e2ba4cd76e530022e..3f3f802506f462246e200d112f832a4ab47a2b27 100644 --- a/src/shared/cplusplus/AST.h +++ b/src/shared/cplusplus/AST.h @@ -805,6 +805,7 @@ public: unsigned rparen_token; SpecifierAST *cv_qualifier_seq; ExceptionSpecificationAST *exception_specification; + ExpressionAST *as_cpp_initializer; public: // annotations Function *symbol; diff --git a/src/shared/cplusplus/CheckDeclarator.cpp b/src/shared/cplusplus/CheckDeclarator.cpp index f1cc0a3f981d754515ea54c6db505f7cd491bca3..ceef0d1d0559e7cce77719b7a1c87cd4f546220f 100644 --- a/src/shared/cplusplus/CheckDeclarator.cpp +++ b/src/shared/cplusplus/CheckDeclarator.cpp @@ -157,6 +157,7 @@ bool CheckDeclarator::visit(NestedDeclaratorAST *ast) bool CheckDeclarator::visit(FunctionDeclaratorAST *ast) { Function *fun = control()->newFunction(ast->firstToken()); + fun->setAmbiguous(ast->as_cpp_initializer != 0); ast->symbol = fun; fun->setReturnType(_fullySpecifiedType); diff --git a/src/shared/cplusplus/Parser.cpp b/src/shared/cplusplus/Parser.cpp index dac159bf22b2c08e973d92b44499fca666b9d080..423a05e05be8352e4ebccc5b24166fbbefc9bfbb 100644 --- a/src/shared/cplusplus/Parser.cpp +++ b/src/shared/cplusplus/Parser.cpp @@ -977,7 +977,7 @@ bool Parser::parseCoreDeclarator(DeclaratorAST *&node) return false; } -bool Parser::parseDeclarator(DeclaratorAST *&node) +bool Parser::parseDeclarator(DeclaratorAST *&node, bool stopAtCppInitializer) { if (! parseCoreDeclarator(node)) return false; @@ -988,6 +988,47 @@ bool Parser::parseDeclarator(DeclaratorAST *&node) unsigned startOfPostDeclarator = cursor(); if (LA() == T_LPAREN) { + if (stopAtCppInitializer) { + unsigned lparen_token = cursor(); + ExpressionAST *initializer = 0; + + bool blocked = blockErrors(true); + if (parseInitializer(initializer)) { + if (NestedExpressionAST *expr = initializer->asNestedExpression()) { + if (expr->expression && expr->rparen_token && (LA() == T_COMMA || LA() == T_SEMICOLON)) { + rewind(lparen_token); + + // check for ambiguous declarators. + + consumeToken(); + ParameterDeclarationClauseAST *parameter_declaration_clause = 0; + if (parseParameterDeclarationClause(parameter_declaration_clause) && LA() == T_RPAREN) { + unsigned rparen_token = consumeToken(); + + FunctionDeclaratorAST *ast = new (_pool) FunctionDeclaratorAST; + ast->lparen_token = lparen_token; + ast->parameters = parameter_declaration_clause; + ast->as_cpp_initializer = initializer; + ast->rparen_token = rparen_token; + *postfix_ptr = ast; + postfix_ptr = &(*postfix_ptr)->next; + + blockErrors(blocked); + return true; + } + + + blockErrors(blocked); + rewind(lparen_token); + return true; + } + } + } + + blockErrors(blocked); + rewind(lparen_token); + } + FunctionDeclaratorAST *ast = new (_pool) FunctionDeclaratorAST; ast->lparen_token = consumeToken(); parseParameterDeclarationClause(ast->parameters); @@ -1494,7 +1535,7 @@ bool Parser::parseInitDeclarator(DeclaratorAST *&node, if (acceptStructDeclarator && LA() == T_COLON) { // anonymous bit-field declaration. // ### TODO create the AST - } else if (! parseDeclarator(node)) { + } else if (! parseDeclarator(node, /*stopAtCppInitializer = */ ! acceptStructDeclarator)) { return false; } diff --git a/src/shared/cplusplus/Parser.h b/src/shared/cplusplus/Parser.h index 638c1147bdd54a54644ae44d19cefb128e7e5dfc..b2d55f3868fe4ac6750740b390054d3700e2a5c9 100644 --- a/src/shared/cplusplus/Parser.h +++ b/src/shared/cplusplus/Parser.h @@ -107,7 +107,7 @@ public: bool parseSimpleDeclaration(DeclarationAST *&node, bool acceptStructDeclarator = false); bool parseDeclarationStatement(StatementAST *&node); bool parseCoreDeclarator(DeclaratorAST *&node); - bool parseDeclarator(DeclaratorAST *&node); + bool parseDeclarator(DeclaratorAST *&node, bool stopAtCppInitializer = false); bool parseDeleteExpression(ExpressionAST *&node); bool parseDoStatement(StatementAST *&node); bool parseElaboratedTypeSpecifier(SpecifierAST *&node); diff --git a/src/shared/cplusplus/Symbols.cpp b/src/shared/cplusplus/Symbols.cpp index 35d2d9c6906739322d5986500a44b05d10c75b13..5185aa82659a8fd1f5e6897f21abfeab6cc4a5d0 100644 --- a/src/shared/cplusplus/Symbols.cpp +++ b/src/shared/cplusplus/Symbols.cpp @@ -255,6 +255,12 @@ bool Function::isPureVirtual() const void Function::setPureVirtual(bool isPureVirtual) { _isPureVirtual = isPureVirtual; } +bool Function::isAmbiguous() const +{ return _isAmbiguous; } + +void Function::setAmbiguous(bool isAmbiguous) +{ _isAmbiguous = isAmbiguous; } + void Function::visitSymbol0(SymbolVisitor *visitor) { if (visitor->visit(this)) { diff --git a/src/shared/cplusplus/Symbols.h b/src/shared/cplusplus/Symbols.h index a9f8cda5479c26f8d26efd5f7951011318517eb4..8efc0fbdbed0c104a843564817f77520b8131223 100644 --- a/src/shared/cplusplus/Symbols.h +++ b/src/shared/cplusplus/Symbols.h @@ -322,6 +322,9 @@ public: virtual Function *asFunctionType() { return this; } + bool isAmbiguous() const; // internal + void setAmbiguous(bool isAmbiguous); // internal + protected: virtual void visitSymbol0(SymbolVisitor *visitor); virtual void accept0(TypeVisitor *visitor); @@ -337,6 +340,7 @@ private: unsigned _isPureVirtual: 1; unsigned _isConst: 1; unsigned _isVolatile: 1; + unsigned _isAmbiguous: 1; unsigned _methodKey: 3; }; };