Use Q_UNUSED instead of commenting out arg names

Signed-off-by: Felix Weilbach <felix.weilbach@nextcloud.com>
nextcloud · Jul 21, 2021 · 3026516 · 3026516
1 parent 3f6bb4a
commit 3026516
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/src/gui/wizard/webview.cpp b/src/gui/wizard/webview.cpp
@@ -218,8 +218,11 @@ bool WebEnginePage::certificateError(const QWebEngineCertificateError &certifica
     return ret == QMessageBox::Yes;
 }
 
-bool WebEnginePage::acceptNavigationRequest(const QUrl &url, QWebEnginePage::NavigationType /*type*/, bool /*isMainFrame*/)
+bool WebEnginePage::acceptNavigationRequest(const QUrl &url, QWebEnginePage::NavigationType type, bool isMainFrame)
 {
+    Q_UNUSED(type);
+    Q_UNUSED(isMainFrame);
+
     if (_enforceHttps && url.scheme() != QStringLiteral("https")) {
         QMessageBox::warning(nullptr, "Security warning", "Can not follow non https link on a https website. This might be a security issue. Please contact your administrator");
         return false;