diff options
| author | Lioncash <mathew1800@gmail.com> | 2022-11-29 08:55:30 -0500 |
|---|---|---|
| committer | Lioncash <mathew1800@gmail.com> | 2022-11-29 08:55:33 -0500 |
| commit | c4af7b3f5cb9a24b48709fcabd21a3cceb2a43c5 (patch) | |
| tree | e0dc3ca4de17440858c1a46ff662372b01ea2e62 /src/video_core/host1x/syncpoint_manager.cpp | |
| parent | 55a3cbfa0db011afa3ad8dd93741dbe87baddab7 (diff) | |
host1x/syncpoint_manager: Pass DeregisterAction() handle as const-ref
The handle is only compared against and not modified in any way, so we
can pass it by const reference.
This also allows us to mark the respective parameters for
DeregisterGuestAction() and DeregisterHostAction() as const references
as well.
Diffstat (limited to 'src/video_core/host1x/syncpoint_manager.cpp')
| -rw-r--r-- | src/video_core/host1x/syncpoint_manager.cpp | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/video_core/host1x/syncpoint_manager.cpp b/src/video_core/host1x/syncpoint_manager.cpp index a44fc83d3..8f23ce527 100644 --- a/src/video_core/host1x/syncpoint_manager.cpp +++ b/src/video_core/host1x/syncpoint_manager.cpp @@ -34,7 +34,7 @@ SyncpointManager::ActionHandle SyncpointManager::RegisterAction( } void SyncpointManager::DeregisterAction(std::list<RegisteredAction>& action_storage, - ActionHandle& handle) { + const ActionHandle& handle) { std::unique_lock lk(guard); // We want to ensure the iterator still exists prior to erasing it @@ -49,11 +49,11 @@ void SyncpointManager::DeregisterAction(std::list<RegisteredAction>& action_stor } } -void SyncpointManager::DeregisterGuestAction(u32 syncpoint_id, ActionHandle& handle) { +void SyncpointManager::DeregisterGuestAction(u32 syncpoint_id, const ActionHandle& handle) { DeregisterAction(guest_action_storage[syncpoint_id], handle); } -void SyncpointManager::DeregisterHostAction(u32 syncpoint_id, ActionHandle& handle) { +void SyncpointManager::DeregisterHostAction(u32 syncpoint_id, const ActionHandle& handle) { DeregisterAction(host_action_storage[syncpoint_id], handle); } |
