diff options
author | Oswald Buddenhagen <oswald.buddenhagen@qt.io> | 2016-12-16 13:34:02 +0100 |
---|---|---|
committer | Oswald Buddenhagen <oswald.buddenhagen@qt.io> | 2018-09-14 13:49:09 +0000 |
commit | 1ec1e7a3b9faf303adbb56d50e52a12579c1a5b4 (patch) | |
tree | 5b9a8829e365eb47f7874d08cd01fba5bb97ad59 | |
parent | 9fc2e29035b15f91be51383dec79406bcc518fff (diff) |
micro-optimize FOO-=$$BAR for empty FOO
there is no point in iterating BAR if FOO is (or became) empty.
Change-Id: I86c89bf0ad726a5ab7ead990a27ef7cc32caebbf
Reviewed-by: Joerg Bornemann <joerg.bornemann@qt.io>
(cherry picked from qtbase/33c33f6475910bad607abac67eef6c581d6188dd)
Reviewed-by: Tor Arne Vestbø <tor.arne.vestbo@qt.io>
-rw-r--r-- | src/linguist/shared/proitems.cpp | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/src/linguist/shared/proitems.cpp b/src/linguist/shared/proitems.cpp index 37ec0c76b..f2745eec8 100644 --- a/src/linguist/shared/proitems.cpp +++ b/src/linguist/shared/proitems.cpp @@ -396,9 +396,12 @@ void ProStringList::removeAll(const char *str) void ProStringList::removeEach(const ProStringList &value) { - for (const ProString &str : value) + for (const ProString &str : value) { + if (isEmpty()) + break; if (!str.isEmpty()) removeAll(str); + } } void ProStringList::removeEmpty() |