From e306254f502e16e163d45d458703899db5c6a7fc Mon Sep 17 00:00:00 2001
From: Roberto Raggi <roberto.raggi@nokia.com>
Date: Fri, 13 Aug 2010 18:16:17 +0200
Subject: Fixed the ranges for the scope regions and process ConditionAST nodes

---
 src/shared/cplusplus/Bind.cpp | 22 +++++++++++++++-------
 1 file changed, 15 insertions(+), 7 deletions(-)

(limited to 'src/shared/cplusplus')

diff --git a/src/shared/cplusplus/Bind.cpp b/src/shared/cplusplus/Bind.cpp
index 9b5fe050ec..54688596af 100644
--- a/src/shared/cplusplus/Bind.cpp
+++ b/src/shared/cplusplus/Bind.cpp
@@ -1178,7 +1178,7 @@ bool Bind::visit(ExpressionStatementAST *ast)
 bool Bind::visit(ForeachStatementAST *ast)
 {
     Block *block = control()->newBlock(ast->firstToken());
-    const unsigned startScopeToken = ast->lparen_token ? ast->lastToken() : ast->firstToken();
+    const unsigned startScopeToken = ast->lparen_token ? ast->lparen_token : ast->firstToken();
     block->setStartOffset(tokenAt(startScopeToken).end());
     block->setEndOffset(tokenAt(ast->lastToken() - 1).end());
     _scope->addMember(block);
@@ -1210,7 +1210,7 @@ bool Bind::visit(ForeachStatementAST *ast)
 bool Bind::visit(ForStatementAST *ast)
 {
     Block *block = control()->newBlock(ast->firstToken());
-    const unsigned startScopeToken = ast->lparen_token ? ast->lastToken() : ast->firstToken();
+    const unsigned startScopeToken = ast->lparen_token ? ast->lparen_token : ast->firstToken();
     block->setStartOffset(tokenAt(startScopeToken).end());
     block->setEndOffset(tokenAt(ast->lastToken() - 1).end());
     _scope->addMember(block);
@@ -1230,7 +1230,7 @@ bool Bind::visit(ForStatementAST *ast)
 bool Bind::visit(IfStatementAST *ast)
 {
     Block *block = control()->newBlock(ast->firstToken());
-    const unsigned startScopeToken = ast->lparen_token ? ast->lastToken() : ast->firstToken();
+    const unsigned startScopeToken = ast->lparen_token ? ast->lparen_token : ast->firstToken();
     block->setStartOffset(tokenAt(startScopeToken).end());
     block->setEndOffset(tokenAt(ast->lastToken() - 1).end());
     _scope->addMember(block);
@@ -1286,7 +1286,7 @@ bool Bind::visit(ReturnStatementAST *ast)
 bool Bind::visit(SwitchStatementAST *ast)
 {
     Block *block = control()->newBlock(ast->firstToken());
-    const unsigned startScopeToken = ast->lparen_token ? ast->lastToken() : ast->firstToken();
+    const unsigned startScopeToken = ast->lparen_token ? ast->lparen_token : ast->firstToken();
     block->setStartOffset(tokenAt(startScopeToken).end());
     block->setEndOffset(tokenAt(ast->lastToken() - 1).end());
     _scope->addMember(block);
@@ -1312,7 +1312,7 @@ bool Bind::visit(TryBlockStatementAST *ast)
 bool Bind::visit(CatchClauseAST *ast)
 {
     Block *block = control()->newBlock(ast->firstToken());
-    const unsigned startScopeToken = ast->lparen_token ? ast->lastToken() : ast->firstToken();
+    const unsigned startScopeToken = ast->lparen_token ? ast->lparen_token : ast->firstToken();
     block->setStartOffset(tokenAt(startScopeToken).end());
     block->setEndOffset(tokenAt(ast->lastToken() - 1).end());
     _scope->addMember(block);
@@ -1329,7 +1329,7 @@ bool Bind::visit(CatchClauseAST *ast)
 bool Bind::visit(WhileStatementAST *ast)
 {
     Block *block = control()->newBlock(ast->firstToken());
-    const unsigned startScopeToken = ast->lparen_token ? ast->lastToken() : ast->firstToken();
+    const unsigned startScopeToken = ast->lparen_token ? ast->lparen_token : ast->firstToken();
     block->setStartOffset(tokenAt(startScopeToken).end());
     block->setEndOffset(tokenAt(ast->lastToken() - 1).end());
     _scope->addMember(block);
@@ -1345,7 +1345,7 @@ bool Bind::visit(WhileStatementAST *ast)
 bool Bind::visit(ObjCFastEnumerationAST *ast)
 {
     Block *block = control()->newBlock(ast->firstToken());
-    const unsigned startScopeToken = ast->lparen_token ? ast->lastToken() : ast->firstToken();
+    const unsigned startScopeToken = ast->lparen_token ? ast->lparen_token : ast->firstToken();
     block->setStartOffset(tokenAt(startScopeToken).end());
     block->setEndOffset(tokenAt(ast->lastToken() - 1).end());
     _scope->addMember(block);
@@ -1444,6 +1444,14 @@ bool Bind::visit(ConditionAST *ast)
     }
     DeclaratorIdAST *declaratorId = 0;
     type = this->declarator(ast->declarator, type, &declaratorId);
+
+    if (declaratorId && declaratorId->name) {
+        unsigned sourceLocation = declaratorId->name->firstToken();
+        Declaration *decl = control()->newDeclaration(sourceLocation, declaratorId->name->name);
+        decl->setType(type);
+        _scope->addMember(decl);
+    }
+
     return false;
 }
 
-- 
cgit v1.2.1