From 8391d926c7355a0ff637c2b604e259d0d25c3cb2 Mon Sep 17 00:00:00 2001 From: Berthold Stoeger Date: Thu, 6 Feb 2020 22:56:10 +0100 Subject: Cleanup: remove const bool parameters and return types These just make no sense. Since the value is copied, it has no meaning to the caller whether the function can change the value (and vice versa for return types). Signed-off-by: Berthold Stoeger --- desktop-widgets/simplewidgets.cpp | 2 +- desktop-widgets/simplewidgets.h | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) (limited to 'desktop-widgets') diff --git a/desktop-widgets/simplewidgets.cpp b/desktop-widgets/simplewidgets.cpp index bba06542f..cf4b9b0ae 100644 --- a/desktop-widgets/simplewidgets.cpp +++ b/desktop-widgets/simplewidgets.cpp @@ -637,7 +637,7 @@ QString TextHyperlinkEventFilter::tryToFormulateUrl(QTextCursor *cursor) return stringMeetsOurUrlRequirements(maybeUrlStr) ? maybeUrlStr : QString(); } -QString TextHyperlinkEventFilter::fromCursorTilWhitespace(QTextCursor *cursor, const bool searchBackwards) +QString TextHyperlinkEventFilter::fromCursorTilWhitespace(QTextCursor *cursor, bool searchBackwards) { // fromCursorTilWhitespace calls cursor->movePosition repeatedly, while // preserving the original 'anchor' (qt terminology) of the cursor. diff --git a/desktop-widgets/simplewidgets.h b/desktop-widgets/simplewidgets.h index bf520422f..99cfc0fa2 100644 --- a/desktop-widgets/simplewidgets.h +++ b/desktop-widgets/simplewidgets.h @@ -156,7 +156,7 @@ private: void handleUrlClick(const QString &urlStr); void handleUrlTooltip(const QString &urlStr, const QPoint &pos); bool stringMeetsOurUrlRequirements(const QString &maybeUrlStr); - QString fromCursorTilWhitespace(QTextCursor *cursor, const bool searchBackwards); + QString fromCursorTilWhitespace(QTextCursor *cursor, bool searchBackwards); QString tryToFormulateUrl(QTextCursor *cursor); QTextEdit const *const textEdit; -- cgit v1.2.3-70-g09d2