diff --git a/Source/Core/Common/Config/Config.h b/Source/Core/Common/Config/Config.h index d7b370282a..ea6058a8d3 100644 --- a/Source/Core/Common/Config/Config.h +++ b/Source/Core/Common/Config/Config.h @@ -29,9 +29,9 @@ void AddLayer(std::unique_ptr loader); std::shared_ptr GetLayer(LayerType layer); void RemoveLayer(LayerType layer); -// Returns an ID that can be passed to RemoveConfigChangedCallback(). -// The callback may be called from any thread. -ConfigChangedCallbackID AddConfigChangedCallback(ConfigChangedCallback func); +// Returns an ID that should be passed to RemoveConfigChangedCallback() when the callback is no +// longer needed. The callback may be called from any thread. +[[nodiscard]] ConfigChangedCallbackID AddConfigChangedCallback(ConfigChangedCallback func); void RemoveConfigChangedCallback(ConfigChangedCallbackID callback_id); void OnConfigChanged(); diff --git a/Source/Core/Core/CPUThreadConfigCallback.h b/Source/Core/Core/CPUThreadConfigCallback.h index 3fb57de89a..03070f76c7 100644 --- a/Source/Core/Core/CPUThreadConfigCallback.h +++ b/Source/Core/Core/CPUThreadConfigCallback.h @@ -18,8 +18,9 @@ struct ConfigChangedCallbackID bool operator==(const ConfigChangedCallbackID&) const = default; }; -// returns an ID that can be passed to RemoveConfigChangedCallback() -ConfigChangedCallbackID AddConfigChangedCallback(Config::ConfigChangedCallback func); +// Returns an ID that should be passed to RemoveConfigChangedCallback() when the callback is no +// longer needed. +[[nodiscard]] ConfigChangedCallbackID AddConfigChangedCallback(Config::ConfigChangedCallback func); void RemoveConfigChangedCallback(ConfigChangedCallbackID callback_id);