Age | Commit message (Collapse) | Author |
|
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!
|
|
ipc_helpers: Only allow trivially copyable objects with PushRaw() and PopRaw()
|
|
aes_util: Allow SetIV() to be non-allocating
|
|
buffer_queue: Make use of designated initializers/std::nullopt where applicable
|
|
These functions are only ever used internally as implementation details
for GenerateRandomRange(), so these can be given internal linkage.
|
|
Resolves a few compiler warnings.
|
|
|
|
Makes our type conversion explicit.
|
|
common/atomic_ops: Don't cast away volatile from pointers
|
|
It's undefined behavior to use non-trivially copyable objects with
std::memcpy, so we can add asserts to catch usages of these at
compile-time.
|
|
We can add a helper function to make creation of these files nicer.
While we're at it, we can eliminate an unnecessary std::array copy in
the constructor. This makes the overhead on some of these functions way
less intensive, given some arrays were quite large.
e.g. The timezone location names are 9633 bytes in size.
|
|
loader: Make IdentifyFile typesafe
|
|
time_zone_binary: Make use of designated initializers
|
|
vi: IApplicationDisplayService:GetIndirectLayerImageRequiredMemoryInfo
|