[network/ruqola] src/widgets/room/delegate: Use Q_UNUSED

Laurent Montel <[email protected]>
Newsgroups gmane.comp.kde.cvs
Message-ID <[email protected]>
Git commit 2537b5cc471bd826dd95f9b9a047e1d724aab2dd by Laurent Montel.
Committed on 31/07/2026 at 11:25.
Pushed by mlaurent into branch 'master'.

Use Q_UNUSED

M  +11   -16   src/widgets/room/delegate/messagedelegatehelpercontext.cpp
M  +12   -20   src/widgets/room/delegate/messagedelegatehelperdivider.cpp
M  +5    -4    src/widgets/room/delegate/messagedelegatehelperurlpreview.cpp

https://invent.kde.org/network/ruqola/-/commit/2537b5cc471bd826dd95f9b9a047e1d724aab2dd

diff --git a/src/widgets/room/delegate/messagedelegatehelpercontext.cpp b/src/widgets/room/delegate/messagedelegatehelpercontext.cpp
index 4ccaaf9818..badb1434fd 100644
--- a/src/widgets/room/delegate/messagedelegatehelpercontext.cpp
+++ b/src/widgets/room/delegate/messagedelegatehelpercontext.cpp
@@ -28,10 +28,9 @@ MessageDelegateHelperContext::~MessageDelegateHelperContext() = default;
 void MessageDelegateHelperContext::draw(const Block &block,
                                         QPainter *painter,
                                         QRect blockRect,
-                                        const QModelIndex &index,
+                                        [[maybe_unused]] const QModelIndex &index,
                                         const QStyleOptionViewItem &option) const
 {
-    Q_UNUSED(index)
     const ContextLayout layout = layoutContext(block, option, blockRect.width());
     for (const auto &button : std::as_const(layout.textList)) {
         const QRectF textRect = button.buttonRect.translated(blockRect.topLeft());
@@ -40,9 +39,11 @@ void MessageDelegateHelperContext::draw(const Block &block,
     }
 }
 
-QSize MessageDelegateHelperContext::sizeHint(const Block &block, const QModelIndex &index, int maxWidth, const QStyleOptionViewItem &option) const
+QSize MessageDelegateHelperContext::sizeHint(const Block &block,
+                                             [[maybe_unused]] const QModelIndex &index,
+                                             int maxWidth,
+                                             const QStyleOptionViewItem &option) const
 {
-    Q_UNUSED(index)
     const ContextLayout layout = layoutContext(block, option, maxWidth);
     if (layout.textList.isEmpty()) {
         return {};
@@ -56,24 +57,18 @@ QSize MessageDelegateHelperContext::sizeHint(const Block &block, const QModelInd
     return {qMax(0, static_cast<int>(width)), height};
 }
 
-bool MessageDelegateHelperContext::handleMouseEvent(const Block &block,
-                                                    QMouseEvent *mouseEvent,
-                                                    QRect blocksRect,
-                                                    const QStyleOptionViewItem &option,
-                                                    const QModelIndex &index)
+bool MessageDelegateHelperContext::handleMouseEvent([[maybe_unused]] const Block &block,
+                                                    [[maybe_unused]] QMouseEvent *mouseEvent,
+                                                    [[maybe_unused]] QRect blocksRect,
+                                                    [[maybe_unused]] const QStyleOptionViewItem &option,
+                                                    [[maybe_unused]] const QModelIndex &index)
 {
-    Q_UNUSED(block);
-    Q_UNUSED(mouseEvent);
-    Q_UNUSED(blocksRect);
-    Q_UNUSED(option);
-    Q_UNUSED(index);
     return false;
 }
 
 MessageDelegateHelperContext::ContextLayout
-MessageDelegateHelperContext::layoutContext(const Block &block, const QStyleOptionViewItem &option, int blockRectWidth) const
+MessageDelegateHelperContext::layoutContext(const Block &block, const QStyleOptionViewItem &option, [[maybe_unused]] int blockRectWidth) const
 {
-    Q_UNUSED(blockRectWidth)
     ContextLayout layout;
 
     qreal x = 0;
diff --git a/src/widgets/room/delegate/messagedelegatehelperdivider.cpp b/src/widgets/room/delegate/messagedelegatehelperdivider.cpp
index d2ab9e2252..85d99ad8dc 100644
--- a/src/widgets/room/delegate/messagedelegatehelperdivider.cpp
+++ b/src/widgets/room/delegate/messagedelegatehelperdivider.cpp
@@ -18,15 +18,12 @@ MessageDelegateHelperDivider::MessageDelegateHelperDivider(RocketChatAccount *ac
 
 MessageDelegateHelperDivider::~MessageDelegateHelperDivider() = default;
 
-void MessageDelegateHelperDivider::draw(const Block &block,
+void MessageDelegateHelperDivider::draw([[maybe_unused]] const Block &block,
                                         QPainter *painter,
                                         QRect blockRect,
-                                        const QModelIndex &index,
-                                        const QStyleOptionViewItem &option) const
+                                        [[maybe_unused]] const QModelIndex &index,
+                                        [[maybe_unused]] const QStyleOptionViewItem &option) const
 {
-    Q_UNUSED(option);
-    Q_UNUSED(block);
-    Q_UNUSED(index)
     // Draw separator
     // painter->fillRect(blockRect, Qt::red);
     const QPen oldPen = painter->pen();
@@ -37,25 +34,20 @@ void MessageDelegateHelperDivider::draw(const Block &block,
     painter->setPen(oldPen);
 }
 
-QSize MessageDelegateHelperDivider::sizeHint(const Block &block, const QModelIndex &index, int maxWidth, const QStyleOptionViewItem &option) const
+QSize MessageDelegateHelperDivider::sizeHint([[maybe_unused]] const Block &block,
+                                             [[maybe_unused]] const QModelIndex &index,
+                                             int maxWidth,
+                                             [[maybe_unused]] const QStyleOptionViewItem &option) const
 {
-    Q_UNUSED(option);
-    Q_UNUSED(block);
-    Q_UNUSED(index)
     const int height = DelegatePaintUtil::margin();
     return {qMax(0, maxWidth), height};
 }
 
-bool MessageDelegateHelperDivider::handleMouseEvent(const Block &block,
-                                                    QMouseEvent *mouseEvent,
-                                                    QRect blocksRect,
-                                                    const QStyleOptionViewItem &option,
-                                                    const QModelIndex &index)
+bool MessageDelegateHelperDivider::handleMouseEvent([[maybe_unused]] const Block &block,
+                                                    [[maybe_unused]] QMouseEvent *mouseEvent,
+                                                    [[maybe_unused]] QRect blocksRect,
+                                                    [[maybe_unused]] const QStyleOptionViewItem &option,
+                                                    [[maybe_unused]] const QModelIndex &index)
 {
-    Q_UNUSED(block);
-    Q_UNUSED(mouseEvent);
-    Q_UNUSED(blocksRect);
-    Q_UNUSED(option);
-    Q_UNUSED(index);
     return false;
 }
diff --git a/src/widgets/room/delegate/messagedelegatehelperurlpreview.cpp b/src/widgets/room/delegate/messagedelegatehelperurlpreview.cpp
index f57330df5c..5b5e5f36cd 100644
--- a/src/widgets/room/delegate/messagedelegatehelperurlpreview.cpp
+++ b/src/widgets/room/delegate/messagedelegatehelperurlpreview.cpp
@@ -104,9 +104,8 @@ void MessageDelegateHelperUrlPreview::draw(const MessageUrl &messageUrl,
 MessageDelegateHelperUrlPreview::PreviewLayout MessageDelegateHelperUrlPreview::layoutPreview(const MessageUrl &messageUrl,
                                                                                               const QStyleOptionViewItem &option,
                                                                                               int urlsPreviewWidth,
-                                                                                              int urlsPreviewHeight) const
+                                                                                              [[maybe_unused]] int urlsPreviewHeight) const
 {
-    Q_UNUSED(urlsPreviewHeight);
     MessageDelegateHelperUrlPreview::PreviewLayout layout;
     layout.isShown = messageUrl.showPreview();
 
@@ -202,9 +201,11 @@ void MessageDelegateHelperUrlPreview::drawDescription(const MessageUrl &messageU
     MessageDelegateUtils::drawSelection(doc, textRect, textRect.top(), painter, index, option, mTextSelectionImpl->textSelection(), {}, messageUrl);
 }
 
-QSize MessageDelegateHelperUrlPreview::sizeHint(const MessageUrl &messageUrl, const QModelIndex &index, int maxWidth, const QStyleOptionViewItem &option) const
+QSize MessageDelegateHelperUrlPreview::sizeHint(const MessageUrl &messageUrl,
+                                                [[maybe_unused]] const QModelIndex &index,
+                                                int maxWidth,
+                                                const QStyleOptionViewItem &option) const
 {
-    Q_UNUSED(index);
     const PreviewLayout layout = layoutPreview(messageUrl, option, maxWidth, -1);
     return {layout.contentWidth, layout.contentHeight};
 }
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.