diff options
| author | 2018-07-24 09:55:15 -0400 | |
|---|---|---|
| committer | 2018-07-24 09:55:17 -0400 | |
| commit | c73410bf2c58a17e0411456ea0ddb6d91872e079 (patch) | |
| tree | edf9d381857d652b1be8edebe22496efa63e3034 /src/core/hle/kernel/svc.cpp | |
| parent | Merge pull request #798 from lioncash/const (diff) | |
| download | yuzu-c73410bf2c58a17e0411456ea0ddb6d91872e079.tar.gz yuzu-c73410bf2c58a17e0411456ea0ddb6d91872e079.tar.xz yuzu-c73410bf2c58a17e0411456ea0ddb6d91872e079.zip | |
svc: Resolve sign comparison warnings in WaitSynchronization()
The loop's induction variable was signed, but we were comparing against
an unsigned variable.
Diffstat (limited to 'src/core/hle/kernel/svc.cpp')
| -rw-r--r-- | src/core/hle/kernel/svc.cpp | 11 |
1 files changed, 7 insertions, 4 deletions
diff --git a/src/core/hle/kernel/svc.cpp b/src/core/hle/kernel/svc.cpp index 7b41c9cfd..da7cacb57 100644 --- a/src/core/hle/kernel/svc.cpp +++ b/src/core/hle/kernel/svc.cpp | |||
| @@ -165,11 +165,14 @@ static ResultCode WaitSynchronization(Handle* index, VAddr handles_address, u64 | |||
| 165 | using ObjectPtr = SharedPtr<WaitObject>; | 165 | using ObjectPtr = SharedPtr<WaitObject>; |
| 166 | std::vector<ObjectPtr> objects(handle_count); | 166 | std::vector<ObjectPtr> objects(handle_count); |
| 167 | 167 | ||
| 168 | for (int i = 0; i < handle_count; ++i) { | 168 | for (u64 i = 0; i < handle_count; ++i) { |
| 169 | Handle handle = Memory::Read32(handles_address + i * sizeof(Handle)); | 169 | const Handle handle = Memory::Read32(handles_address + i * sizeof(Handle)); |
| 170 | auto object = g_handle_table.Get<WaitObject>(handle); | 170 | const auto object = g_handle_table.Get<WaitObject>(handle); |
| 171 | if (object == nullptr) | 171 | |
| 172 | if (object == nullptr) { | ||
| 172 | return ERR_INVALID_HANDLE; | 173 | return ERR_INVALID_HANDLE; |
| 174 | } | ||
| 175 | |||
| 173 | objects[i] = object; | 176 | objects[i] = object; |
| 174 | } | 177 | } |
| 175 | 178 | ||