| Age | Commit message (Collapse) | Author |
|---|
|
We can query the given object name directly from the widget itself. This
removes any potential for forgetting to change the name if the widget
gets renamed and makes the API much simpler (just pass in the widget,
and not worry about its name).
|
|
Eliminates a verbose function cast.
|
|
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.
|
|
We need to provide a message for this variant of the macro, so we can
simply log out the type being used.
|
|
This was assigning the field to itself, which is a no-op. The size
doesn't change between its initial assignment and this one, so this is a
safe change to make.
|
|
Given this is implicitly creating a std::optional, we can move the
vector into it.
|
|
Allows the compiler to avoid unnecessarily zeroing out the internal
buffer of std::optional on some implementations.
|
|
Prevents pessimization of the move constructor (which thankfully didn't
actually happen in practice here, given std::thread isn't copyable).
|
|
Semicolons after a function definition aren't necessary.
|
|
|
|
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
|
|
gl_shader_cache: Use std::max() for determining num_workers
|
|
General: Tidy up clang-format warnings part 2
|
|
|
|
Does not allocate more threads than available in the host system for boot-time shader compilation and always allocates at least 1 thread if hardware_concurrency() returns 0.
|
|
freezer: Make use of std::erase_if
|
|
common/virtual_buffer: drop unused includes
|
|
General: Tidy up clang-format warnings
|
|
There were two issues with block linear copies. First the swizzling was
wrong and this commit reimplements them.
The other issue was that these copies are generally used to download
render targets from the GPU and yuzu was not downloading them from
host GPU memory unless the extreme GPU accuracy setting was selected.
This commit enables cached memory reads for all accuracy levels.
- Fixes level thumbnails in Super Mario Maker 2.
|
|
cheat_engine: Make use of designated initializers
|
|
This PR is now only the Analog devices handling the range value defaulting at 100%
|
|
|
|
|
|
|
|
|
|
kernel: Remove unused variables
|
|
vfs_vector: Make creation of array vfs files less verbose
|
|
|
|
game_list_worker: Do not clear entries when > 1 gamedir is present
|
|
am: Unstub SetScreenShotPermission
|
|
video_core: Remove redundant pixel format type
|
|
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()
|
