diff --git a/src/plugins/vcsbase/baseannotationhighlighter.cpp b/src/plugins/vcsbase/baseannotationhighlighter.cpp index ac23e85394f..6c455515177 100644 --- a/src/plugins/vcsbase/baseannotationhighlighter.cpp +++ b/src/plugins/vcsbase/baseannotationhighlighter.cpp @@ -93,20 +93,21 @@ BaseAnnotationHighlighter::~BaseAnnotationHighlighter() void BaseAnnotationHighlighter::setChangeNumbers(const ChangeNumbers &changeNumbers) { d->m_changeNumberMap.clear(); - if (!changeNumbers.isEmpty()) { - // Assign a color gradient to annotation change numbers. Give - // each change number a unique color. - const QList colors = - TextEditor::SyntaxHighlighter::generateColors(changeNumbers.size(), d->m_background); - int m = 0; - const int cstep = colors.count() / changeNumbers.count(); - const ChangeNumbers::const_iterator cend = changeNumbers.constEnd(); - for (ChangeNumbers::const_iterator it = changeNumbers.constBegin(); it != cend; ++it) { - QTextCharFormat format; - format.setForeground(colors.at(m)); - d->m_changeNumberMap.insert(*it, format); - m += cstep; - } + const int changeNumberCount = changeNumbers.size(); + if (changeNumberCount == 0) + return; + // Assign a color gradient to annotation change numbers. Give + // each change number a unique color. + const QList colors = + TextEditor::SyntaxHighlighter::generateColors(changeNumberCount, d->m_background); + int m = 0; + const int cstep = colors.count() / changeNumberCount; + const ChangeNumbers::const_iterator cend = changeNumbers.constEnd(); + for (ChangeNumbers::const_iterator it = changeNumbers.constBegin(); it != cend; ++it) { + QTextCharFormat format; + format.setForeground(colors.at(m)); + d->m_changeNumberMap.insert(*it, format); + m += cstep; } }