diff options
| author | Lioncash <mathew1800@gmail.com> | 2019-04-01 17:59:43 -0400 |
|---|---|---|
| committer | Lioncash <mathew1800@gmail.com> | 2019-04-01 17:59:45 -0400 |
| commit | 2d70c30fb2628b4cc3a23b6fda34c2211085b905 (patch) | |
| tree | f949f512154d4e282fc28deb03cd8df8b1ac738f /src | |
| parent | 26d03811615e8d021435691c7ae599ac169792ef (diff) | |
kernel/thread: Avoid sign conversion within GetCommandBufferAddress()
Previously this was performing a u64 + int sign conversion. When dealing
with addresses, we should generally be keeping the arithmetic in the
same signedness type.
This also gets rid of the static lifetime of the constant, as there's no
need to make a trivial type like this potentially live for the entire
duration of the program.
Diffstat (limited to 'src')
| -rw-r--r-- | src/core/hle/kernel/thread.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/core/hle/kernel/thread.cpp b/src/core/hle/kernel/thread.cpp index 937b41767..332f96287 100644 --- a/src/core/hle/kernel/thread.cpp +++ b/src/core/hle/kernel/thread.cpp @@ -237,8 +237,8 @@ s32 Thread::GetWaitObjectIndex(const WaitObject* object) const { VAddr Thread::GetCommandBufferAddress() const { // Offset from the start of TLS at which the IPC command buffer begins. - static constexpr int CommandHeaderOffset = 0x80; - return GetTLSAddress() + CommandHeaderOffset; + constexpr u64 command_header_offset = 0x80; + return GetTLSAddress() + command_header_offset; } void Thread::SetStatus(ThreadStatus new_status) { |