| Age | Commit message (Collapse) | Author |
|---|
|
vfs_real: Avoid redundant map lookups
|
|
We can just log out the parameters in the meantime.
|
|
Same behavior, minus unnecessary inclusions where not necessary.
|
|
Initialize id to a deterministic value and also mark the unreachable
cases in the switch with UNREACHABLE().
|
|
Simplifies code.
|
|
Removes the need for comments to indicate the fields being assigned.
|
|
core_timing: Resolve sign conversion warning
|
|
We can convert these maps into constexpr arrays to eliminate some
runtime static constructors.
|
|
We can simplify a few loops by making use of ranged for.
|
|
|
|
Same behavior, minus unnecessary zeroing out of the pointer.
|
|
This constant is only ever assigned to downcount, which is a s64, not a
u64.
|
|
dynarmic: Add unsafe optimizations
|
|
Migrates the Telemetry namespace into the Common namespace to make the
code consistent with the rest of our common code.
|
|
|
|
Avoids some trivially avoidable map lookups by keeping the result of
find operations around and querying them.
|
|
common/fileutil: Convert namespace to Common::FS
|
|
aes_util: Make use of non-template variant of Transcode
|
|
|
|
Migrates a remaining common file over to the Common namespace, making it
consistent with the rest of common files.
This also allows for high-traffic FS related code to alias the
filesystem function namespace as
namespace FS = Common::FS;
for more concise typing.
|
|
core: Resolve several -Wextra-semi warnings
|
|
software_keyboard: Resolve a pessimizing move warning
|
|
system_control: Make functions internally linked where applicable
|
|
kernel/scheduler: Mark SchedulerLock constructor as nodiscard
|
|
Prevents a useless self-assignment from occurring.
|
|
A std::vector created in place like this is already an rvalue and
doesn't need to be moved.
|
|
We can amend one of the cascade macros to require semicolons in order to
compile. In other cases, we can just remove the superfluous semicolons.
|
|
Ensures that callers make use of the constructor, preventing bugs from
silently occurring.
|
|
Allows the compiler to warn about cases where the constructor is used
but then immediately discarded, which is a potential cause of
locking/unlocking bugs.
|
|
cheat_engine: Resolve implicit bool->u64 conversion
|
|
General: Tidy up clang-format warnings part 2
|
|
|
|
freezer: Make use of std::erase_if
|
|
cheat_engine: Make use of designated initializers
|
|
kernel: Remove unused variables
|
|
vfs_vector: Make creation of array vfs files less verbose
|
|
am: Unstub SetScreenShotPermission
|
|
This makes it more inline with its currently unavailable standardized
analogue std::derived_from.
While we're at it, we can also make the template match the requirements
of the standardized variant as well.
|
|
partition_data_manager: Make data arrays constexpr
|
|
address_arbiter/scheduler: Resolve sign conversion warnings
|
|
Cleans up the callsites in other functions.
|
|
VAddr will always be 64-bit, so there's no need to take a trivial
primitive alias by reference.
|
|
With C++20 we can simplify the erasing idiom.
|
|
We can just return zero here.
|
|
Same behavior, but makes the member being assigned obvious.
|
|
Fills in some hashes that were previously unhandled.
|
|
Previously the constructor for all of these would run at program
startup, consuming time before the application can enter main().
This is also particularly dangerous, given the logging system wouldn't
have been initialized properly yet, yet the program would use the logs
to signify an error.
To rectify this, we can replace the literals with constexpr functions
that perform the conversion at compile-time, completely eliminating the
runtime cost of initializing these arrays.
|
|
We can use sizeof to make it obvious at the call site where the value is
coming from.
|
|
Same behavior, less template instantiations.
|
|
- In `SetCurrentThreadName`, when on Linux, truncate to 15 bytes, as (at
least on glibc) `pthread_set_name_np` will otherwise return `ERANGE` and
do nothing.
- Also, add logging in case `pthread_set_name_np` returns an error
anyway. This is Linux-specific, as the Apple and BSD versions of
`pthread_set_name_np return `void`.
- Change the name for CPU threads in multi-core mode from
"yuzu:CoreCPUThread_N" (19 bytes) to "yuzu:CPUCore_N" (14 bytes) so it
fits into the Linux limit. Some other thread names are also cut off,
but I didn't bother addressing them as you can guess them from the
truncated versions. For a CPU thread, truncation means you can't see
which core it is!
|
