@@ -29,7 +29,7 @@ typedef QPair<Declaration*, int> DeclarationDepthPair;
 
 namespace Python {
 
-QList<CompletionTreeItemPointer> PythonCodeCompletionContext::completionItems(bool& abort, bool fullCompletion)
+QList<CompletionTreeItemPointer> PythonCodeCompletionContext::completionItems(bool& /*abort*/, bool /*fullCompletion*/)
 {
     QList<CompletionTreeItemPointer> items;
     DUChainReadLocker lock(DUChain::lock());
 
@@ -17,7 +17,7 @@ PythonCodeCompletionWorker::PythonCodeCompletionWorker(PythonCodeCompletionModel
 
 }
 
-KDevelop::CodeCompletionContext* PythonCodeCompletionWorker::createCompletionContext(KDevelop::DUContextPointer context, const QString& contextText, const QString& followingText, const KDevelop::CursorInRevision& position) const
+KDevelop::CodeCompletionContext* PythonCodeCompletionWorker::createCompletionContext(KDevelop::DUContextPointer context, const QString& contextText, const QString& /*followingText*/, const KDevelop::CursorInRevision& position) const
 {
     PythonCodeCompletionContext* completionContext = new PythonCodeCompletionContext(context, contextText, position, 0);
     return completionContext;
 
@@ -103,7 +103,7 @@ void ContextBuilder::setEditor(PythonEditorIntegrator* editor)
     ContextBuilder::m_editor = editor;
 }
 
-void ContextBuilder::setEditor(ParseSession* session)
+void ContextBuilder::setEditor(ParseSession* /*session*/)
 {
     PythonEditorIntegrator* e = new PythonEditorIntegrator(/*session*/);
     //m_identifierCompiler = new IdentifierCompiler(e->parseSession());
@@ -147,7 +147,7 @@ void ContextBuilder::addImportedContexts()
     }
 }
 
-void ContextBuilder::openContextForStatementList( const QList<Ast*>& l, DUContext::ContextType type)
+void ContextBuilder::openContextForStatementList( const QList<Ast*>& l, DUContext::ContextType /*type*/)
 {
     if ( l.count() > 0 )
     {
@@ -229,7 +229,7 @@ void ContextBuilder::visitImport(ImportAst* node)
 {
     foreach ( AliasAst* name, node->names ) {
         // for "import ... as", use the as thingy, use the module name otherwise
-        Identifier* variableDeclarationName = name->asName ? name->asName->identifier : name->name;
+//         Identifier* variableDeclarationName = name->asName ? name->asName->identifier : name->name; # TODO check this
 
         KUrl moduleFilePath = findModulePath(name->name->value);
         if ( ! moduleFilePath.isValid() ) continue;
 
@@ -249,7 +249,7 @@ void DeclarationBuilder::visitAssignment(AssignmentAst* node)
 void DeclarationBuilder::visitClassDefinition( ClassDefinitionAst* node )
 {
     kDebug() << "opening class definition";
-    ClassDeclaration* classDec = new ClassDeclaration(editorFindRange(node->body.first(), node->body.last()), currentContext());
+//     ClassDeclaration* classDec = new ClassDeclaration(editorFindRange(node->body.first(), node->body.last()), currentContext());
 
     openDeclaration<ClassDeclaration>( node->name, node );
     eventuallyAssignInternalContext();
 
@@ -136,7 +136,7 @@ ExceptionHandlerAst::ExceptionHandlerAst(Ast* parent): Ast(parent, Ast::Exceptio
 
 }
 
-ExecAst::ExecAst(Ast* parent): StatementAst(parent, Ast::ExecAstType), body(0), locals(0), globals(0)
+ExecAst::ExecAst(Ast* parent): StatementAst(parent, Ast::ExecAstType), body(0), globals(0), locals(0)
 {
 
 }
 
@@ -138,7 +138,7 @@ CodeAst* AstBuilder::parseXmlAst(QString xml)
     return codeAst;
 }
 
-void AstBuilder::parseXmlAstNode(QXmlStreamReader* xmlast, QXmlStreamReader::TokenType token = QXmlStreamReader::Invalid) {
+void AstBuilder::parseXmlAstNode(QXmlStreamReader* xmlast, QXmlStreamReader::TokenType /*token = QXmlStreamReader::Invalid*/) {
     bool nodeAdded = false;
 
     while ( ! xmlast->atEnd() && ! xmlast->hasError() ) {
@@ -165,7 +165,7 @@ void AstBuilder::parseXmlAstNode(QXmlStreamReader* xmlast, QXmlStreamReader::Tok
             }
 
             // this will push a parent onto the stack
-            nodeAdded = parseAstNode(currentElementName, currentElementText, currentElementAttributes);
+            nodeAdded = parseAstNode(currentElementName, /*currentElementText,*/ currentElementAttributes); // we might need ElementText some day
             if ( ! nodeAdded ) {
                 m_isRealNodeMap.append(false);
                 continue;
@@ -198,7 +198,7 @@ void AstBuilder::parseXmlAstNode(QXmlStreamReader* xmlast, QXmlStreamReader::Tok
     }
 }
 
-bool AstBuilder::parseAstNode(QString name, QString text, const QList< QXmlStreamAttribute >& attributes)
+bool AstBuilder::parseAstNode(QString name, /*QString text, */ const QList< QXmlStreamAttribute >& attributes)
 {
     Ast* ast;
 
 
@@ -53,7 +53,7 @@ class AstBuilder
     CodeAst* parseXmlAst(QString xml);
     QString getXmlForFile(KUrl filename, const QString& contents);
     void parseXmlAstNode(QXmlStreamReader* xmlast, QXmlStreamReader::TokenType token);
-    bool parseAstNode(QString name, QString text, const QList<QXmlStreamAttribute>& attributes);
+    bool parseAstNode(QString name, /*QString text, */const QList<QXmlStreamAttribute>& attributes);
 
     KDevelop::TopDUContext* m_topContext;
 
 
@@ -70,7 +70,7 @@ void Driver::setCurrentDocument(KUrl url)
     m_currentDocument = url;
 }
 
-QPair<CodeAst*, bool> Driver::parse( Python::CodeAst* ast )
+QPair<CodeAst*, bool> Driver::parse( Python::CodeAst* /* ast */)
 {
     AstBuilder pythonparser;
     QPair<CodeAst*, bool> matched;
 
@@ -148,18 +148,18 @@ void ParseJob::run()
         m_duContext = builder.build(filename, m_ast);
         setDuChain(m_duContext);
 
+        UseBuilder usebuilder( &editor );
+        usebuilder.buildUses(m_ast);
+        
         {
             DUChainWriteLocker lock(DUChain::lock());
+            m_duContext->clearProblems();
             ParsingEnvironmentFilePointer parsingEnvironmentFile = m_duContext->parsingEnvironmentFile();
             parsingEnvironmentFile->clearModificationRevisions();
             parsingEnvironmentFile->setModificationRevision(contents().modification);
             DUChain::self()->updateContextEnvironment(m_duContext, parsingEnvironmentFile.data());
-            m_duContext->clearProblems();
         }
 
-        UseBuilder usebuilder( &editor );
-        usebuilder.buildUses(m_ast);
-        
         kDebug() << "----Parsing Succeded---***";
 
         if ( m_parent && m_parent->codeHighlighting() ) {
@@ -181,21 +181,27 @@ void ParseJob::run()
             static const IndexedString langString("python");
             file->setLanguage(langString);
             m_duContext = new TopDUContext(document(), RangeInRevision(0, 0, INT_MAX, INT_MAX), file);
-            DUChain::self()->addDocumentChain(m_duContext);
+//             DUChain::self()->addDocumentChain(m_duContext);
         }
         {
             m_duContext->parsingEnvironmentFile()->clearModificationRevisions();
             m_duContext->parsingEnvironmentFile()->setModificationRevision(contents().modification);
             m_duContext->clearProblems();
             DUChain::self()->updateContextEnvironment(m_duContext, m_duContext->parsingEnvironmentFile().data());
         }
-
+        
         foreach ( ProblemPointer p, m_session->m_problems ) {
             kDebug() << "Added problem to context";
             m_duContext->addProblem(p);
         }
         setDuChain(m_duContext);
     }
+    
+    DUChainWriteLocker lock(DUChain::lock());
+    if ( ! DUChain::self()->chainForDocument(document()) && m_duContext ) {
+        DUChain::self()->addDocumentChain(m_duContext);
+    }
+    
 }
 
 ParseSession *ParseJob::parseSession() const
Original file line number	Diff line number	Diff line change
`@@ -29,7 +29,7 @@ typedef QPair<Declaration*, int> DeclarationDepthPair;`
`29`	`29`
`30`	`30`	`namespace Python {`
`31`	`31`
`32`		`-QList<CompletionTreeItemPointer> PythonCodeCompletionContext::completionItems(bool& abort, bool fullCompletion)`
	`32`	`+QList<CompletionTreeItemPointer> PythonCodeCompletionContext::completionItems(bool& /abort/, bool /fullCompletion/)`
`33`	`33`	`{`
`34`	`34`	`QList<CompletionTreeItemPointer> items;`
`35`	`35`	`DUChainReadLocker lock(DUChain::lock());`
Original file line number	Diff line number	Diff line change
`@@ -17,7 +17,7 @@ PythonCodeCompletionWorker::PythonCodeCompletionWorker(PythonCodeCompletionModel`
`17`	`17`
`18`	`18`	`}`
`19`	`19`
`20`		`-KDevelop::CodeCompletionContext* PythonCodeCompletionWorker::createCompletionContext(KDevelop::DUContextPointer context, const QString& contextText, const QString& followingText, const KDevelop::CursorInRevision& position) const`
	`20`	`+KDevelop::CodeCompletionContext* PythonCodeCompletionWorker::createCompletionContext(KDevelop::DUContextPointer context, const QString& contextText, const QString& /followingText/, const KDevelop::CursorInRevision& position) const`
`21`	`21`	`{`
`22`	`22`	`PythonCodeCompletionContext* completionContext = new PythonCodeCompletionContext(context, contextText, position, 0);`
`23`	`23`	`return completionContext;`
Original file line number	Diff line number	Diff line change
`@@ -103,7 +103,7 @@ void ContextBuilder::setEditor(PythonEditorIntegrator* editor)`
`103`	`103`	`ContextBuilder::m_editor = editor;`
`104`	`104`	`}`
`105`	`105`
`106`		`-void ContextBuilder::setEditor(ParseSession* session)`
	`106`	`+void ContextBuilder::setEditor(ParseSession* /session/)`
`107`	`107`	`{`
`108`	`108`	`PythonEditorIntegrator* e = new PythonEditorIntegrator(/session/);`
`109`	`109`	`//m_identifierCompiler = new IdentifierCompiler(e->parseSession());`
`@@ -147,7 +147,7 @@ void ContextBuilder::addImportedContexts()`
`147`	`147`	`}`
`148`	`148`	`}`
`149`	`149`
`150`		`-void ContextBuilder::openContextForStatementList( const QList<Ast*>& l, DUContext::ContextType type)`
	`150`	`+void ContextBuilder::openContextForStatementList( const QList<Ast>& l, DUContext::ContextType /type*/)`
`151`	`151`	`{`
`152`	`152`	`if ( l.count() > 0 )`
`153`	`153`	`{`
`@@ -229,7 +229,7 @@ void ContextBuilder::visitImport(ImportAst* node)`
`229`	`229`	`{`
`230`	`230`	`foreach ( AliasAst* name, node->names ) {`
`231`	`231`	`// for "import ... as", use the as thingy, use the module name otherwise`
`232`		`- Identifier* variableDeclarationName = name->asName ? name->asName->identifier : name->name;`
	`232`	`+// Identifier* variableDeclarationName = name->asName ? name->asName->identifier : name->name; # TODO check this`
`233`	`233`
`234`	`234`	`KUrl moduleFilePath = findModulePath(name->name->value);`
`235`	`235`	`if ( ! moduleFilePath.isValid() ) continue;`
Original file line number	Diff line number	Diff line change
`@@ -249,7 +249,7 @@ void DeclarationBuilder::visitAssignment(AssignmentAst* node)`
`249`	`249`	`void DeclarationBuilder::visitClassDefinition( ClassDefinitionAst* node )`
`250`	`250`	`{`
`251`	`251`	`kDebug() << "opening class definition";`
`252`		`- ClassDeclaration* classDec = new ClassDeclaration(editorFindRange(node->body.first(), node->body.last()), currentContext());`
	`252`	`+// ClassDeclaration* classDec = new ClassDeclaration(editorFindRange(node->body.first(), node->body.last()), currentContext());`
`253`	`253`
`254`	`254`	`openDeclaration<ClassDeclaration>( node->name, node );`
`255`	`255`	`eventuallyAssignInternalContext();`
Original file line number	Diff line number	Diff line change
`@@ -136,7 +136,7 @@ ExceptionHandlerAst::ExceptionHandlerAst(Ast* parent): Ast(parent, Ast::Exceptio`
`136`	`136`
`137`	`137`	`}`
`138`	`138`
`139`		`-ExecAst::ExecAst(Ast* parent): StatementAst(parent, Ast::ExecAstType), body(0), locals(0), globals(0)`
	`139`	`+ExecAst::ExecAst(Ast* parent): StatementAst(parent, Ast::ExecAstType), body(0), globals(0), locals(0)`
`140`	`140`	`{`
`141`	`141`
`142`	`142`	`}`
Original file line number	Diff line number	Diff line change
`@@ -138,7 +138,7 @@ CodeAst* AstBuilder::parseXmlAst(QString xml)`
`138`	`138`	`return codeAst;`
`139`	`139`	`}`
`140`	`140`
`141`		`-void AstBuilder::parseXmlAstNode(QXmlStreamReader* xmlast, QXmlStreamReader::TokenType token = QXmlStreamReader::Invalid) {`
	`141`	`+void AstBuilder::parseXmlAstNode(QXmlStreamReader* xmlast, QXmlStreamReader::TokenType /token = QXmlStreamReader::Invalid/) {`
`142`	`142`	`bool nodeAdded = false;`
`143`	`143`
`144`	`144`	`while ( ! xmlast->atEnd() && ! xmlast->hasError() ) {`
`@@ -165,7 +165,7 @@ void AstBuilder::parseXmlAstNode(QXmlStreamReader* xmlast, QXmlStreamReader::Tok`
`165`	`165`	`}`
`166`	`166`
`167`	`167`	`// this will push a parent onto the stack`
`168`		`- nodeAdded = parseAstNode(currentElementName, currentElementText, currentElementAttributes);`
	`168`	`+ nodeAdded = parseAstNode(currentElementName, /currentElementText,/ currentElementAttributes); // we might need ElementText some day`
`169`	`169`	`if ( ! nodeAdded ) {`
`170`	`170`	`m_isRealNodeMap.append(false);`
`171`	`171`	`continue;`
`@@ -198,7 +198,7 @@ void AstBuilder::parseXmlAstNode(QXmlStreamReader* xmlast, QXmlStreamReader::Tok`
`198`	`198`	`}`
`199`	`199`	`}`
`200`	`200`
`201`		`-bool AstBuilder::parseAstNode(QString name, QString text, const QList< QXmlStreamAttribute >& attributes)`
	`201`	`+bool AstBuilder::parseAstNode(QString name, /QString text, / const QList< QXmlStreamAttribute >& attributes)`
`202`	`202`	`{`
`203`	`203`	`Ast* ast;`
`204`	`204`
Original file line number	Diff line number	Diff line change
`@@ -70,7 +70,7 @@ void Driver::setCurrentDocument(KUrl url)`
`70`	`70`	`m_currentDocument = url;`
`71`	`71`	`}`
`72`	`72`
`73`		`-QPair<CodeAst, bool> Driver::parse( Python::CodeAst ast )`
	`73`	`+QPair<CodeAst, bool> Driver::parse( Python::CodeAst /* ast */)`
`74`	`74`	`{`
`75`	`75`	`AstBuilder pythonparser;`
`76`	`76`	`QPair<CodeAst*, bool> matched;`