From d89f4d9cdbc167112e64fca1931e81bc27d17319 Mon Sep 17 00:00:00 2001 From: Manuel Nickschas Date: Mon, 10 Sep 2018 00:04:49 +0200 Subject: [PATCH] modernize: Use 'while(true)' instead of 'while(1)' --- src/client/messagemodel.cpp | 2 +- src/qtui/chatviewsearchcontroller.cpp | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/client/messagemodel.cpp b/src/client/messagemodel.cpp index 8eae52bb..d817babc 100644 --- a/src/client/messagemodel.cpp +++ b/src/client/messagemodel.cpp @@ -349,7 +349,7 @@ int MessageModel::indexForId(MsgId id) // binary search int start = 0; int end = messageCount() - 1; - while (1) { + while (true) { if (end - start == 1) return end; int pivot = (end + start) / 2; diff --git a/src/qtui/chatviewsearchcontroller.cpp b/src/qtui/chatviewsearchcontroller.cpp index 23609136..e5026cd7 100644 --- a/src/qtui/chatviewsearchcontroller.cpp +++ b/src/qtui/chatviewsearchcontroller.cpp @@ -145,7 +145,7 @@ void ChatViewSearchController::updateHighlights(bool reuse) int start = 0; int end = _highlightItems.count() - 1; QPointF startPos; QPointF endPos; - while (1) { + while (true) { startPos = _highlightItems[start]->scenePos(); endPos = _highlightItems[end]->scenePos(); if (startPos == oldHighlightPos) { -- 2.20.1