summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorRoberto Raggi <roberto.raggi@nokia.com>2010-07-16 18:00:54 +0200
committerRoberto Raggi <roberto.raggi@nokia.com>2010-07-19 10:32:25 +0200
commit615381a5b12449438cfd0f546e043f7a7d8c0609 (patch)
tree4500624c2e13ea4c89f0d92d970b487fb6e5f247
parent5871f2bb8181b39357cffa5677a75424151ca494 (diff)
downloadqt-creator-615381a5b12449438cfd0f546e043f7a7d8c0609.tar.gz
Use addUsage() instead of addTypeUsage().
-rw-r--r--src/plugins/cppeditor/cppchecksymbols.cpp36
-rw-r--r--src/plugins/cppeditor/cppchecksymbols.h8
2 files changed, 22 insertions, 22 deletions
diff --git a/src/plugins/cppeditor/cppchecksymbols.cpp b/src/plugins/cppeditor/cppchecksymbols.cpp
index 1fb655758e..cdf8162510 100644
--- a/src/plugins/cppeditor/cppchecksymbols.cpp
+++ b/src/plugins/cppeditor/cppchecksymbols.cpp
@@ -353,7 +353,7 @@ bool CheckSymbols::visit(NamespaceAST *ast)
unsigned line, column;
getTokenStartPosition(ast->identifier_token, &line, &column);
Use use(line, column, tok.length());
- addTypeUsage(use);
+ addUsage(use);
}
}
@@ -409,7 +409,7 @@ void CheckSymbols::checkName(NameAST *ast)
if (_potentialTypes.contains(id)) {
Scope *scope = findScope(ast);
const QList<LookupItem> candidates = _context.lookup(ast->name, scope);
- addTypeUsage(candidates, ast);
+ addUsage(candidates, ast);
} else if (_potentialMembers.contains(id)) {
Scope *scope = findScope(ast);
const QList<LookupItem> candidates = _context.lookup(ast->name, scope);
@@ -470,7 +470,7 @@ bool CheckSymbols::visit(QualifiedNameAST *ast)
const Name *name = class_or_namespace_name->name;
b = _context.lookupType(name, scope);
- addTypeUsage(b, class_or_namespace_name);
+ addUsage(b, class_or_namespace_name);
for (it = it->next; b && it; it = it->next) {
NestedNameSpecifierAST *nested_name_specifier = it->value;
@@ -480,14 +480,14 @@ bool CheckSymbols::visit(QualifiedNameAST *ast)
accept(template_id->template_argument_list);
b = b->findType(class_or_namespace_name->name);
- addTypeUsage(b, class_or_namespace_name);
+ addUsage(b, class_or_namespace_name);
}
}
}
}
if (b && ast->unqualified_name)
- addTypeUsage(b->find(ast->unqualified_name->name), ast->unqualified_name);
+ addUsage(b->find(ast->unqualified_name->name), ast->unqualified_name);
}
return false;
@@ -501,7 +501,7 @@ bool CheckSymbols::visit(TypenameTypeParameterAST *ast)
if (_potentialTypes.contains(id)) {
Scope *scope = findScope(_templateDeclarationStack.back());
const QList<LookupItem> candidates = _context.lookup(ast->name->name, scope);
- addTypeUsage(candidates, ast->name);
+ addUsage(candidates, ast->name);
}
}
}
@@ -538,7 +538,7 @@ bool CheckSymbols::visit(FunctionDefinitionAST *ast)
QList<SemanticInfo::Use> uses;
foreach (uses, locals.uses) {
foreach (const SemanticInfo::Use &u, uses)
- addTypeUsage(u);
+ addUsage(u);
}
_functionDefinitionStack.removeLast();
@@ -547,10 +547,10 @@ bool CheckSymbols::visit(FunctionDefinitionAST *ast)
return false;
}
-void CheckSymbols::addTypeUsage(const Use &use)
+void CheckSymbols::addUsage(const Use &use)
{
if (_functionDefinitionStack.isEmpty()) {
- if (_typeUsages.size() >= 50) {
+ if (_usages.size() >= 50) {
if (_flushRequested && use.line != _flushLine)
flush();
else if (! _flushRequested) {
@@ -560,10 +560,10 @@ void CheckSymbols::addTypeUsage(const Use &use)
}
}
- _typeUsages.append(use);
+ _usages.append(use);
}
-void CheckSymbols::addTypeUsage(ClassOrNamespace *b, NameAST *ast)
+void CheckSymbols::addUsage(ClassOrNamespace *b, NameAST *ast)
{
if (! b)
return;
@@ -580,11 +580,11 @@ void CheckSymbols::addTypeUsage(ClassOrNamespace *b, NameAST *ast)
getTokenStartPosition(startToken, &line, &column);
const unsigned length = tok.length();
const Use use(line, column, length);
- addTypeUsage(use);
+ addUsage(use);
//qDebug() << "added use" << oo(ast->name) << line << column << length;
}
-void CheckSymbols::addTypeUsage(const QList<LookupItem> &candidates, NameAST *ast)
+void CheckSymbols::addUsage(const QList<LookupItem> &candidates, NameAST *ast)
{
unsigned startToken = ast->firstToken();
if (DestructorNameAST *dtor = ast->asDestructorName())
@@ -608,7 +608,7 @@ void CheckSymbols::addTypeUsage(const QList<LookupItem> &candidates, NameAST *as
c->isClass() || c->isEnum() ||
c->isForwardClassDeclaration() || c->isTypenameArgument()) {
const Use use(line, column, length);
- addTypeUsage(use);
+ addUsage(use);
//qDebug() << "added use" << oo(ast->name) << line << column << length;
break;
}
@@ -643,7 +643,7 @@ void CheckSymbols::addMemberUsage(const QList<LookupItem> &candidates, NameAST *
continue;
const Use use(line, column, length, Use::Field);
- addTypeUsage(use);
+ addUsage(use);
//qDebug() << "added use" << oo(ast->name) << line << column << length;
}
}
@@ -684,9 +684,9 @@ void CheckSymbols::flush()
_flushRequested = false;
_flushLine = 0;
- if (_typeUsages.isEmpty())
+ if (_usages.isEmpty())
return;
- reportResults(_typeUsages);
- _typeUsages.clear();
+ reportResults(_usages);
+ _usages.clear();
}
diff --git a/src/plugins/cppeditor/cppchecksymbols.h b/src/plugins/cppeditor/cppchecksymbols.h
index 8db7fda967..43029dbb82 100644
--- a/src/plugins/cppeditor/cppchecksymbols.h
+++ b/src/plugins/cppeditor/cppchecksymbols.h
@@ -83,9 +83,9 @@ protected:
void checkName(NameAST *ast);
void checkNamespace(NameAST *name);
- void addTypeUsage(ClassOrNamespace *b, NameAST *ast);
- void addTypeUsage(const QList<LookupItem> &candidates, NameAST *ast);
- void addTypeUsage(const Use &use);
+ void addUsage(ClassOrNamespace *b, NameAST *ast);
+ void addUsage(const QList<LookupItem> &candidates, NameAST *ast);
+ void addUsage(const Use &use);
void checkMemberName(NameAST *ast);
void addMemberUsage(const QList<LookupItem> &candidates, NameAST *ast);
@@ -126,7 +126,7 @@ private:
QList<ScopedSymbol *> _scopes;
QList<TemplateDeclarationAST *> _templateDeclarationStack;
QList<FunctionDefinitionAST *> _functionDefinitionStack;
- QVector<Use> _typeUsages;
+ QVector<Use> _usages;
bool _flushRequested;
unsigned _flushLine;
};