Starting to get rid of Q_UNUSED declarations for unused function parameters.

The official C++ way is to leave out the variable name instead. This has the
advantage to be reliable: In one case a variable declared unused was later
used.

Change-Id: I1c96636f7fa7a621d1594d1e9cacaec75c561faa
Reviewed-on: https://code.wireshark.org/review/9532
Reviewed-by: Jörg Mayer <jmayer@loplof.de>

1 files changed, 2 insertions, 4 deletions

diff --git a/ui/qt/packet_format_group_box.cpp b/ui/qt/packet_format_group_box.cpp
index d62c55d31a..fbc424d1f2 100644
--- a/ui/qt/packet_format_group_box.cpp
+++ b/ui/qt/packet_format_group_box.cpp
@@ -75,9 +75,8 @@ bool PacketFormatGroupBox::allExpandedEnabled()
     return pf_ui_->allExpandedButton->isChecked();
 }
 
-void PacketFormatGroupBox::on_summaryCheckBox_toggled(bool checked)
+void PacketFormatGroupBox::on_summaryCheckBox_toggled(bool)
 {
-    Q_UNUSED(checked);
     emit formatChanged();
 }
 
@@ -89,9 +88,8 @@ void PacketFormatGroupBox::on_detailsCheckBox_toggled(bool checked)
     emit formatChanged();
 }
 
-void PacketFormatGroupBox::on_bytesCheckBox_toggled(bool checked)
+void PacketFormatGroupBox::on_bytesCheckBox_toggled(bool)
 {
-    Q_UNUSED(checked);
     emit formatChanged();
 }