Use QString::splitRef() instead of QString::split()

Avoid string allocation where it seems feasible.

Change-Id: I61b23d4ef8a459f5aa77727a75f4e1d2b140da3b
Reviewed-by: Tobias Hunger <tobias.hunger@qt.io>

1 files changed, 2 insertions, 2 deletions

diff --git a/src/plugins/texteditor/generichighlighter/highlighter.cpp b/src/plugins/texteditor/generichighlighter/highlighter.cpp
index 877767f4c4..89150ac6d4 100644
--- a/src/plugins/texteditor/generichighlighter/highlighter.cpp
+++ b/src/plugins/texteditor/generichighlighter/highlighter.cpp
@@ -415,8 +415,8 @@ void Highlighter::changeContext(const QString &contextName,
                                 const bool assignCurrent)
 {
     if (contextName.startsWith(kPop)) {
-        QStringList list = contextName.split(kHash, QString::SkipEmptyParts);
-        for (int i = 0; i < list.size(); ++i) {
+        const int count = contextName.splitRef(kHash, QString::SkipEmptyParts).size();
+        for (int i = 0; i < count; ++i) {
             if (m_contexts.isEmpty()) {
                 throw HighlighterException(
                         QCoreApplication::translate("GenericHighlighter", "Reached empty context."));