From c95511eb5f8786754b9926adae5be1e56780c181 Mon Sep 17 00:00:00 2001 From: tt2468 Date: Sun, 21 Nov 2021 03:08:06 -0800 Subject: [PATCH] Base: Remove UNUSED_PARAMETER() usages I prefer to use undefined parameters instead. --- src/WebSocketApi.cpp | 8 ++--- src/eventhandler/EventHandler_Inputs.cpp | 4 +-- src/eventhandler/EventHandler_Scenes.cpp | 4 +-- src/eventhandler/EventHandler_Transitions.cpp | 4 +-- src/forms/ConnectInfo.cpp | 3 +- src/forms/SettingsDialog.cpp | 8 ++--- src/requesthandler/RequestHandler_Config.cpp | 16 +++------- src/requesthandler/RequestHandler_General.cpp | 16 +++------- src/requesthandler/RequestHandler_Record.cpp | 32 +++++-------------- src/requesthandler/RequestHandler_Scenes.cpp | 12 ++----- src/requesthandler/RequestHandler_Stream.cpp | 16 +++------- src/utils/Obs.cpp | 8 ++--- 12 files changed, 33 insertions(+), 98 deletions(-) diff --git a/src/WebSocketApi.cpp b/src/WebSocketApi.cpp index a20f7155..f2e0a1eb 100644 --- a/src/WebSocketApi.cpp +++ b/src/WebSocketApi.cpp @@ -114,7 +114,7 @@ void WebSocketApi::vendor_register_cb(void *priv_data, calldata_t *cd) RETURN_SUCCESS(); } -void WebSocketApi::vendor_request_register_cb(void *priv_data, calldata_t *cd) +void WebSocketApi::vendor_request_register_cb(void *, calldata_t *cd) { Vendor *v = get_vendor(cd); if (!v) @@ -146,11 +146,9 @@ void WebSocketApi::vendor_request_register_cb(void *priv_data, calldata_t *cd) blog_debug("[WebSocketApi::vendor_request_register_cb] [vendorName: %s] Registered new vendor request: %s", v->_name.c_str(), requestType); RETURN_SUCCESS(); - - UNUSED_PARAMETER(priv_data); } -void WebSocketApi::vendor_request_unregister_cb(void *priv_data, calldata_t *cd) +void WebSocketApi::vendor_request_unregister_cb(void *, calldata_t *cd) { Vendor *v = get_vendor(cd); if (!v) @@ -174,8 +172,6 @@ void WebSocketApi::vendor_request_unregister_cb(void *priv_data, calldata_t *cd) blog_debug("[WebSocketApi::vendor_request_unregister_cb] [vendorName: %s] Unregistered vendor request: %s", v->_name.c_str(), requestType); RETURN_SUCCESS(); - - UNUSED_PARAMETER(priv_data); } void WebSocketApi::vendor_event_emit_cb(void *priv_data, calldata_t *cd) diff --git a/src/eventhandler/EventHandler_Inputs.cpp b/src/eventhandler/EventHandler_Inputs.cpp index a198ccb4..01e50199 100644 --- a/src/eventhandler/EventHandler_Inputs.cpp +++ b/src/eventhandler/EventHandler_Inputs.cpp @@ -41,14 +41,12 @@ void EventHandler::HandleInputRemoved(obs_source_t *source) BroadcastEvent(EventSubscription::Inputs, "InputRemoved", eventData); } -void EventHandler::HandleInputNameChanged(obs_source_t *source, std::string oldInputName, std::string inputName) +void EventHandler::HandleInputNameChanged(obs_source_t *, std::string oldInputName, std::string inputName) { json eventData; eventData["oldInputName"] = oldInputName; eventData["inputName"] = inputName; BroadcastEvent(EventSubscription::Inputs, "InputNameChanged", eventData); - - UNUSED_PARAMETER(source); } void EventHandler::HandleInputActiveStateChanged(void *param, calldata_t *data) diff --git a/src/eventhandler/EventHandler_Scenes.cpp b/src/eventhandler/EventHandler_Scenes.cpp index 75eb505f..05c0f587 100644 --- a/src/eventhandler/EventHandler_Scenes.cpp +++ b/src/eventhandler/EventHandler_Scenes.cpp @@ -35,14 +35,12 @@ void EventHandler::HandleSceneRemoved(obs_source_t *source) BroadcastEvent(EventSubscription::Scenes, "SceneRemoved", eventData); } -void EventHandler::HandleSceneNameChanged(obs_source_t *source, std::string oldSceneName, std::string sceneName) +void EventHandler::HandleSceneNameChanged(obs_source_t *, std::string oldSceneName, std::string sceneName) { json eventData; eventData["oldSceneName"] = oldSceneName; eventData["sceneName"] = sceneName; BroadcastEvent(EventSubscription::Scenes, "SceneNameChanged", eventData); - - UNUSED_PARAMETER(source); } void EventHandler::HandleCurrentSceneChanged() diff --git a/src/eventhandler/EventHandler_Transitions.cpp b/src/eventhandler/EventHandler_Transitions.cpp index dfc79472..2ae8d85c 100644 --- a/src/eventhandler/EventHandler_Transitions.cpp +++ b/src/eventhandler/EventHandler_Transitions.cpp @@ -35,12 +35,10 @@ void EventHandler::HandleTransitionRemoved(obs_source_t *source) BroadcastEvent(EventSubscription::Transitions, "TransitionRemoved", eventData); } -void EventHandler::HandleTransitionNameChanged(obs_source_t *source, std::string oldTransitionName, std::string transitionName) +void EventHandler::HandleTransitionNameChanged(obs_source_t *, std::string oldTransitionName, std::string transitionName) { json eventData; eventData["oldTransitionName"] = oldTransitionName; eventData["transitionName"] = transitionName; BroadcastEvent(EventSubscription::Transitions, "TransitionNameChanged", eventData); - - UNUSED_PARAMETER(source); } diff --git a/src/forms/ConnectInfo.cpp b/src/forms/ConnectInfo.cpp index d604762f..1d2317f2 100644 --- a/src/forms/ConnectInfo.cpp +++ b/src/forms/ConnectInfo.cpp @@ -47,9 +47,8 @@ ConnectInfo::~ConnectInfo() delete ui; } -void ConnectInfo::showEvent(QShowEvent *event) +void ConnectInfo::showEvent(QShowEvent *) { - UNUSED_PARAMETER(event); RefreshData(); } diff --git a/src/forms/SettingsDialog.cpp b/src/forms/SettingsDialog.cpp index c32c7f58..abac2b3c 100644 --- a/src/forms/SettingsDialog.cpp +++ b/src/forms/SettingsDialog.cpp @@ -75,7 +75,7 @@ SettingsDialog::~SettingsDialog() delete sessionTableTimer; } -void SettingsDialog::showEvent(QShowEvent *event) +void SettingsDialog::showEvent(QShowEvent *) { auto conf = GetConfig(); if (!conf) { @@ -107,18 +107,14 @@ void SettingsDialog::showEvent(QShowEvent *event) FillSessionTable(); sessionTableTimer->start(1000); - - UNUSED_PARAMETER(event); } -void SettingsDialog::hideEvent(QHideEvent *event) +void SettingsDialog::hideEvent(QHideEvent *) { if (sessionTableTimer->isActive()) sessionTableTimer->stop(); connectInfo->hide(); - - UNUSED_PARAMETER(event); } void SettingsDialog::ToggleShowHide() diff --git a/src/requesthandler/RequestHandler_Config.cpp b/src/requesthandler/RequestHandler_Config.cpp index ada9ee1f..e5b1e60e 100644 --- a/src/requesthandler/RequestHandler_Config.cpp +++ b/src/requesthandler/RequestHandler_Config.cpp @@ -78,14 +78,12 @@ RequestResult RequestHandler::SetPersistentData(const Request& request) return RequestResult::Success(); } -RequestResult RequestHandler::GetSceneCollectionList(const Request& request) +RequestResult RequestHandler::GetSceneCollectionList(const Request&) { json responseData; responseData["currentSceneCollectionName"] = Utils::Obs::StringHelper::GetCurrentSceneCollection(); responseData["sceneCollections"] = Utils::Obs::ListHelper::GetSceneCollectionList(); return RequestResult::Success(responseData); - - UNUSED_PARAMETER(request); } // Does not return until collection has finished switching @@ -135,14 +133,12 @@ RequestResult RequestHandler::CreateSceneCollection(const Request& request) return RequestResult::Success(); } -RequestResult RequestHandler::GetProfileList(const Request& request) +RequestResult RequestHandler::GetProfileList(const Request&) { json responseData; responseData["currentProfileName"] = Utils::Obs::StringHelper::GetCurrentProfile(); responseData["profiles"] = Utils::Obs::ListHelper::GetProfileList(); return RequestResult::Success(responseData); - - UNUSED_PARAMETER(request); } RequestResult RequestHandler::SetCurrentProfile(const Request& request) @@ -267,7 +263,7 @@ RequestResult RequestHandler::SetProfileParameter(const Request& request) return RequestResult::Success(); } -RequestResult RequestHandler::GetVideoSettings(const Request& request) +RequestResult RequestHandler::GetVideoSettings(const Request&) { struct obs_video_info ovi; if (!obs_get_video_info(&ovi)) @@ -282,8 +278,6 @@ RequestResult RequestHandler::GetVideoSettings(const Request& request) responseData["outputHeight"] = ovi.output_height; return RequestResult::Success(responseData); - - UNUSED_PARAMETER(request); } RequestResult RequestHandler::SetVideoSettings(const Request& request) @@ -332,7 +326,7 @@ RequestResult RequestHandler::SetVideoSettings(const Request& request) return RequestResult::Error(RequestStatus::MissingRequestParameter, "You must specify at least one video-changing pair."); } -RequestResult RequestHandler::GetStreamServiceSettings(const Request& request) +RequestResult RequestHandler::GetStreamServiceSettings(const Request&) { json responseData; @@ -342,8 +336,6 @@ RequestResult RequestHandler::GetStreamServiceSettings(const Request& request) responseData["streamServiceSettings"] = Utils::Json::ObsDataToJson(serviceSettings, true); return RequestResult::Success(responseData); - - UNUSED_PARAMETER(request); } RequestResult RequestHandler::SetStreamServiceSettings(const Request& request) diff --git a/src/requesthandler/RequestHandler_General.cpp b/src/requesthandler/RequestHandler_General.cpp index b74b2c09..6e902d20 100644 --- a/src/requesthandler/RequestHandler_General.cpp +++ b/src/requesthandler/RequestHandler_General.cpp @@ -24,7 +24,7 @@ with this program. If not, see #include "../eventhandler/types/EventSubscription.h" #include "../obs-websocket.h" -RequestResult RequestHandler::GetVersion(const Request& request) +RequestResult RequestHandler::GetVersion(const Request&) { json responseData; responseData["obsVersion"] = Utils::Obs::StringHelper::GetObsVersion(); @@ -40,8 +40,6 @@ RequestResult RequestHandler::GetVersion(const Request& request) responseData["supportedImageFormats"] = supportedImageFormats; return RequestResult::Success(responseData); - - UNUSED_PARAMETER(request); } RequestResult RequestHandler::BroadcastCustomEvent(const Request& request) @@ -60,7 +58,7 @@ RequestResult RequestHandler::BroadcastCustomEvent(const Request& request) return RequestResult::Success(); } -RequestResult RequestHandler::GetStats(const Request& request) +RequestResult RequestHandler::GetStats(const Request&) { json responseData = Utils::Obs::DataHelper::GetStats(); @@ -68,17 +66,13 @@ RequestResult RequestHandler::GetStats(const Request& request) responseData["webSocketSessionOutgoingMessages"] = _session->OutgoingMessages(); return RequestResult::Success(responseData); - - UNUSED_PARAMETER(request); } -RequestResult RequestHandler::GetHotkeyList(const Request& request) +RequestResult RequestHandler::GetHotkeyList(const Request&) { json responseData; responseData["hotkeys"] = Utils::Obs::ListHelper::GetHotkeyNameList(); return RequestResult::Success(responseData); - - UNUSED_PARAMETER(request); } RequestResult RequestHandler::TriggerHotkeyByName(const Request& request) @@ -141,13 +135,11 @@ RequestResult RequestHandler::TriggerHotkeyByKeySequence(const Request& request) return RequestResult::Success(); } -RequestResult RequestHandler::GetStudioModeEnabled(const Request& request) +RequestResult RequestHandler::GetStudioModeEnabled(const Request&) { json responseData; responseData["studioModeEnabled"] = obs_frontend_preview_program_mode_active(); return RequestResult::Success(responseData); - - UNUSED_PARAMETER(request); } RequestResult RequestHandler::SetStudioModeEnabled(const Request& request) diff --git a/src/requesthandler/RequestHandler_Record.cpp b/src/requesthandler/RequestHandler_Record.cpp index 7479a7c1..19a2c14a 100644 --- a/src/requesthandler/RequestHandler_Record.cpp +++ b/src/requesthandler/RequestHandler_Record.cpp @@ -19,7 +19,7 @@ with this program. If not, see #include "RequestHandler.h" -RequestResult RequestHandler::GetRecordStatus(const Request& request) +RequestResult RequestHandler::GetRecordStatus(const Request&) { OBSOutputAutoRelease recordOutput = obs_frontend_get_streaming_output(); @@ -33,11 +33,9 @@ RequestResult RequestHandler::GetRecordStatus(const Request& request) responseData["outputBytes"] = (uint64_t)obs_output_get_total_bytes(recordOutput); return RequestResult::Success(responseData); - - UNUSED_PARAMETER(request); } -RequestResult RequestHandler::ToggleRecord(const Request& request) +RequestResult RequestHandler::ToggleRecord(const Request&) { json responseData; if (obs_frontend_recording_active()) { @@ -49,11 +47,9 @@ RequestResult RequestHandler::ToggleRecord(const Request& request) } return RequestResult::Success(responseData); - - UNUSED_PARAMETER(request); } -RequestResult RequestHandler::StartRecord(const Request& request) +RequestResult RequestHandler::StartRecord(const Request&) { if (obs_frontend_recording_active()) return RequestResult::Error(RequestStatus::OutputRunning); @@ -62,11 +58,9 @@ RequestResult RequestHandler::StartRecord(const Request& request) obs_frontend_recording_start(); return RequestResult::Success(); - - UNUSED_PARAMETER(request); } -RequestResult RequestHandler::StopRecord(const Request& request) +RequestResult RequestHandler::StopRecord(const Request&) { if (!obs_frontend_recording_active()) return RequestResult::Error(RequestStatus::OutputNotRunning); @@ -75,11 +69,9 @@ RequestResult RequestHandler::StopRecord(const Request& request) obs_frontend_recording_stop(); return RequestResult::Success(); - - UNUSED_PARAMETER(request); } -RequestResult RequestHandler::ToggleRecordPause(const Request& request) +RequestResult RequestHandler::ToggleRecordPause(const Request&) { json responseData; if (obs_frontend_recording_paused()) { @@ -91,11 +83,9 @@ RequestResult RequestHandler::ToggleRecordPause(const Request& request) } return RequestResult::Success(responseData); - - UNUSED_PARAMETER(request); } -RequestResult RequestHandler::PauseRecord(const Request& request) +RequestResult RequestHandler::PauseRecord(const Request&) { if (obs_frontend_recording_paused()) return RequestResult::Error(RequestStatus::OutputPaused); @@ -104,11 +94,9 @@ RequestResult RequestHandler::PauseRecord(const Request& request) obs_frontend_recording_pause(true); return RequestResult::Success(); - - UNUSED_PARAMETER(request); } -RequestResult RequestHandler::ResumeRecord(const Request& request) +RequestResult RequestHandler::ResumeRecord(const Request&) { if (!obs_frontend_recording_paused()) return RequestResult::Error(RequestStatus::OutputNotPaused); @@ -117,16 +105,12 @@ RequestResult RequestHandler::ResumeRecord(const Request& request) obs_frontend_recording_pause(false); return RequestResult::Success(); - - UNUSED_PARAMETER(request); } -RequestResult RequestHandler::GetRecordDirectory(const Request& request) +RequestResult RequestHandler::GetRecordDirectory(const Request&) { json responseData; responseData["recordDirectory"] = Utils::Obs::StringHelper::GetCurrentRecordOutputPath(); return RequestResult::Success(responseData); - - UNUSED_PARAMETER(request); } diff --git a/src/requesthandler/RequestHandler_Scenes.cpp b/src/requesthandler/RequestHandler_Scenes.cpp index 2661be77..44ec05b4 100644 --- a/src/requesthandler/RequestHandler_Scenes.cpp +++ b/src/requesthandler/RequestHandler_Scenes.cpp @@ -19,7 +19,7 @@ with this program. If not, see #include "RequestHandler.h" -RequestResult RequestHandler::GetSceneList(const Request& request) +RequestResult RequestHandler::GetSceneList(const Request&) { json responseData; @@ -38,19 +38,15 @@ RequestResult RequestHandler::GetSceneList(const Request& request) responseData["scenes"] = Utils::Obs::ListHelper::GetSceneList(); return RequestResult::Success(responseData); - - UNUSED_PARAMETER(request); } -RequestResult RequestHandler::GetCurrentProgramScene(const Request& request) +RequestResult RequestHandler::GetCurrentProgramScene(const Request&) { json responseData; OBSSourceAutoRelease currentProgramScene = obs_frontend_get_current_scene(); responseData["currentProgramSceneName"] = obs_source_get_name(currentProgramScene); return RequestResult::Success(responseData); - - UNUSED_PARAMETER(request); } RequestResult RequestHandler::SetCurrentProgramScene(const Request& request) @@ -66,7 +62,7 @@ RequestResult RequestHandler::SetCurrentProgramScene(const Request& request) return RequestResult::Success(); } -RequestResult RequestHandler::GetCurrentPreviewScene(const Request& request) +RequestResult RequestHandler::GetCurrentPreviewScene(const Request&) { if (!obs_frontend_preview_program_mode_active()) return RequestResult::Error(RequestStatus::StudioModeNotActive); @@ -77,8 +73,6 @@ RequestResult RequestHandler::GetCurrentPreviewScene(const Request& request) responseData["currentPreviewSceneName"] = obs_source_get_name(currentPreviewScene); return RequestResult::Success(responseData); - - UNUSED_PARAMETER(request); } RequestResult RequestHandler::SetCurrentPreviewScene(const Request& request) diff --git a/src/requesthandler/RequestHandler_Stream.cpp b/src/requesthandler/RequestHandler_Stream.cpp index fa5cce3e..8136dcfb 100644 --- a/src/requesthandler/RequestHandler_Stream.cpp +++ b/src/requesthandler/RequestHandler_Stream.cpp @@ -19,7 +19,7 @@ with this program. If not, see #include "RequestHandler.h" -RequestResult RequestHandler::GetStreamStatus(const Request& request) +RequestResult RequestHandler::GetStreamStatus(const Request&) { OBSOutputAutoRelease streamOutput = obs_frontend_get_streaming_output(); @@ -35,11 +35,9 @@ RequestResult RequestHandler::GetStreamStatus(const Request& request) responseData["outputTotalFrames"] = obs_output_get_total_frames(streamOutput); return RequestResult::Success(responseData); - - UNUSED_PARAMETER(request); } -RequestResult RequestHandler::ToggleStream(const Request& request) +RequestResult RequestHandler::ToggleStream(const Request&) { json responseData; if (obs_frontend_streaming_active()) { @@ -51,11 +49,9 @@ RequestResult RequestHandler::ToggleStream(const Request& request) } return RequestResult::Success(responseData); - - UNUSED_PARAMETER(request); } -RequestResult RequestHandler::StartStream(const Request& request) +RequestResult RequestHandler::StartStream(const Request&) { if (obs_frontend_streaming_active()) return RequestResult::Error(RequestStatus::OutputRunning); @@ -64,11 +60,9 @@ RequestResult RequestHandler::StartStream(const Request& request) obs_frontend_streaming_start(); return RequestResult::Success(); - - UNUSED_PARAMETER(request); } -RequestResult RequestHandler::StopStream(const Request& request) +RequestResult RequestHandler::StopStream(const Request&) { if (!obs_frontend_streaming_active()) return RequestResult::Error(RequestStatus::OutputNotRunning); @@ -77,6 +71,4 @@ RequestResult RequestHandler::StopStream(const Request& request) obs_frontend_streaming_stop(); return RequestResult::Success(); - - UNUSED_PARAMETER(request); } diff --git a/src/utils/Obs.cpp b/src/utils/Obs.cpp index 8b206b08..a0d87601 100644 --- a/src/utils/Obs.cpp +++ b/src/utils/Obs.cpp @@ -252,14 +252,12 @@ std::vector Utils::Obs::ListHelper::GetHotkeyList() { std::vector ret; - obs_enum_hotkeys([](void* data, obs_hotkey_id id, obs_hotkey_t* hotkey) { + obs_enum_hotkeys([](void* data, obs_hotkey_id, obs_hotkey_t* hotkey) { auto ret = reinterpret_cast *>(data); ret->push_back(hotkey); return true; - - UNUSED_PARAMETER(id); }, &ret); return ret; @@ -301,7 +299,7 @@ std::vector Utils::Obs::ListHelper::GetSceneItemList(obs_scene_t *scene, b std::pair, bool> enumData; enumData.second = basic; - obs_scene_enum_items(scene, [](obs_scene_t* scene, obs_sceneitem_t* sceneItem, void* param) { + obs_scene_enum_items(scene, [](obs_scene_t*, obs_sceneitem_t* sceneItem, void* param) { auto enumData = reinterpret_cast, bool>*>(param); json item; @@ -325,8 +323,6 @@ std::vector Utils::Obs::ListHelper::GetSceneItemList(obs_scene_t *scene, b enumData->first.push_back(item); return true; - - UNUSED_PARAMETER(scene); }, &enumData); return enumData.first;