diff options
author | hjk <hjk@qt.io> | 2019-07-24 18:40:10 +0200 |
---|---|---|
committer | hjk <hjk@qt.io> | 2019-07-26 09:23:48 +0000 |
commit | 7ab6783e24c6a05a67f319817cd1bdd026a7ce43 (patch) | |
tree | 8b56ea311d333f45f300b915c3bd25a2b77b4aef /src/plugins/cpptools/searchsymbols.cpp | |
parent | eab0df22f98fab37585e4513de836a06e4aa05d5 (diff) |
Standardize on int for line and column values
Recently tons of warnings show up for presumably "problematic"
singned <-> unsigned and size conversions.
The Qt side uses 'int', and that's the biggest 'integration surface'
for us, so instead of establishing some internal boundary between
signed and unsigned areas, push that boundary out of creator core code,
and use 'int' everywhere.
Because it reduces friction further, also do it in libcplusplus.
Change-Id: I84f3b79852c8029713e7ea6f133ffb9ef7030a70
Reviewed-by: Nikolai Kosjar <nikolai.kosjar@qt.io>
Diffstat (limited to 'src/plugins/cpptools/searchsymbols.cpp')
-rw-r--r-- | src/plugins/cpptools/searchsymbols.cpp | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/src/plugins/cpptools/searchsymbols.cpp b/src/plugins/cpptools/searchsymbols.cpp index 6adb437bf80..05080fdb870 100644 --- a/src/plugins/cpptools/searchsymbols.cpp +++ b/src/plugins/cpptools/searchsymbols.cpp @@ -70,7 +70,7 @@ IndexItem::Ptr SearchSymbols::operator()(Document::Ptr doc, const QString &scope QTC_ASSERT(_parent->fileName() == Internal::StringTable::insert(doc->fileName()), return IndexItem::Ptr()); - for (unsigned i = 0, ei = doc->globalSymbolCount(); i != ei; ++i) + for (int i = 0, ei = doc->globalSymbolCount(); i != ei; ++i) accept(doc->globalSymbolAt(i)); Internal::StringTable::scheduleGC(); @@ -95,7 +95,7 @@ bool SearchSymbols::visit(Enum *symbol) QString newScope = scopedSymbolName(name, symbol); ScopedScope scopeRaii(_scope, newScope); - for (unsigned i = 0, ei = symbol->memberCount(); i != ei; ++i) + for (int i = 0, ei = symbol->memberCount(); i != ei; ++i) accept(symbol->memberAt(i)); return false; @@ -112,7 +112,7 @@ bool SearchSymbols::visit(Namespace *symbol) QString name = scopedSymbolName(symbol); QString newScope = name; ScopedScope raii(_scope, newScope); - for (unsigned i = 0; i < symbol->memberCount(); ++i) { + for (int i = 0; i < symbol->memberCount(); ++i) { accept(symbol->memberAt(i)); } return false; @@ -321,7 +321,7 @@ void SearchSymbols::processClass(T *clazz) QString newScope = scopedSymbolName(name, clazz); ScopedScope scopeRaii(_scope, newScope); - for (unsigned i = 0, ei = clazz->memberCount(); i != ei; ++i) + for (int i = 0, ei = clazz->memberCount(); i != ei; ++i) accept(clazz->memberAt(i)); } |