| Age | Commit message (Collapse) | Author |
|
The loop's induction variable was signed, but we were comparing against
an unsigned variable.
|
|
Provides slightly more context than only logging out the address value.
|
|
arm_dynarmic: Make MakeJit() a const member function
|
|
core: Make converting constructors explicit where applicable
|
|
apm/interface: Remove redundant declaration of InstallInterfaces()
|
|
mutex: Pass SharedPtr to GetHighestPriorityMutexWaitingThread() by reference
|
|
We can use one instance of the interface instead of duplicating code.
|
|
|
|
|
|
|
|
|
|
ipc_helpers: Make member variables of ResponseBuilder private
|
|
partition_filesystem: Use std::move where applicable
|
|
This functions doesn't modify instance state, so it can be a made a
const member function.
|
|
Avoids unwanted implicit conversions. Thankfully, given the large amount
of cleanup in past PRs, only this tiny amount is left over to cover.
|
|
This is already declared in apm/apm.h
|
|
The pointed to thread's members are simply observed in this case, so we
don't need to copy it here.
|
|
* Regression and Mode Fixes
* Review Fixes
* string_view correction
* Add operator& for FileSys::Mode
* Return std::string from SanitizePath
* Farming Simulator Fix
* Use != With mode operator&
|
|
|
|
These aren't used externally at all, so they can be made private.
|
|
exclusive_monitor: Use consistent type alias for u64
|
|
loader: Minor cleanup
|
|
linker: Remove unused parameter from WriteRelocations()
|
|
loader/nro: Minor changes
|
|
vi: Minor changes
|
|
hle: Remove unused config_mem and shared_page source files
|
|
Uses the same type aliases we use for virtual addresses, and converts
one lingering usage of std::array<uint64_t, 2> to u128 for consistency.
|
|
Avoids copying a std::string instance and avoids unnecessary atomic
reference count incrementing and decrementing.
|
|
RealVfsFile inherits from VfsFile, the instance from std::make_shared is
already compatible with the function argument type, making the copy
constructor call unnecessary.
|
|
is_jump_relocation is never used within the function, so we can just
remove it.
|
|
It's sufficient to use a forward declaration instead of a direct
inclusion here.
|
|
Makes the code more uniform, and also braces cases where the body of an
unbraced conditional travels more than one line.
|
|
Makes it consistent with the other Apploader constructors, and prevents
implicit conversions.
|
|
This isn't used anywhere in the header.
|
|
NRO Assets and NACP File Format
|
|
It's undefined behavior to memcpy an object that isn't considered
trivially copyable, so put a compile-time check in to make sure this
doesn't occur.
|
|
Allows avoiding unnecessary copies of the vector depending on the
calling code.
While we're at it, remove a redundant no-parameter base constructor call
|
|
This is just an unused hold-over from citra, so we can get rid of this
to trim off an exposed global, among other things.
|
|
This is a holdover from citra that's essentially unused.
|
|
Forgot to include this in 22f448b6327044076959e338811ee576f3dcf093
|
|
Cleanup
Review fixes
|
|
set: Amend return value of GetAvailableLanguageCodes()
|
|
This just returns the size of the language code buffer.
|
|
The return code should be 32-bit in size.
|
|
Kernel/SVC: Perform atomic accesses in SignalProcessWideKey as per the real kernel.
|
|
file_util, vfs: Use std::string_view where applicable
|
|
kernel.
|
|
|
|
ReplaceFileWithSubdirectory() takes a VirtualFile and a VirtualDir, but
it was being passed a string as one of its arguments. The only reason
this never caused issues is because this template isn't instantiated
anywhere yet.
This corrects an issue before it occurs.
|
|
Avoids unnecessary construction of std::string instances where
applicable.
|