| Age | Commit message (Collapse) | Author |
|---|
|
video_core: Remove unnecessary enum class casting in logging messages
|
|
core: Mark unused fields as [[maybe_unused]]
|
|
hle: Type check ResponseBuilder::Push arguments, and fix use in vi.cpp
|
|
Same behavior, but elides an unnecessary atomic reference count
increment and decrement.
|
|
fmt now automatically prints the numeric value of an enum class member
by default, so we don't need to use casts any more.
Reduces the line noise a bit.
|
|
nvdrv: Remove useless re-declaration of pure virtual methods that were already declared in the superclass
|
|
boxcat: Avoid unnecessary object copy
|
|
renderer_vulkan: Add missing `override` specifier
|
|
video_core: Resolve more variable shadowing scenarios pt.3
|
|
Follows our established coding style.
|
|
Removes a documentation comment for a non-existent member.
|
|
This is unused and removing it shrinks the structure by 3584 bytes.
|
|
Since this is inside a string literal, backslashes that are part of
regex syntax have to be escaped. But that's ugly, so convert to a raw
string instead.
|
|
`enum class PollEvents`
Actually, two enum classes, since for some reason there are two separate
yet identical `PollFD` types used in the codebase. I get that one is
ABI-compatible with the Switch while the other is an abstract type used
for the host, but why not use `WSAPOLLFD` directly for the latter?
Anyway, why make this change? Because on Apple platforms, `POLL_IN`,
`POLL_OUT`, etc. (with an underscore) are defined as macros in
<sys/signal.h>. (This is inherited from FreeBSD.) So defining
a variable with the same name causes a compile error.
I could just rename the variables, but while I was at it I thought I
might as well switch to an enum for stronger typing.
Also, change the type used for values copied directly to/from the
`events` and `revents` fields of the host *native*
`pollfd`/`WSASPOLLFD`, from `u32` to `short`, as `short` is the correct
canonical type on both Unix and Windows.
|
|
Fix "explicitly defaulted but implicitly deleted" warning
|
|
`PhysicalCore`'s move assignment operator was declared as `= default`,
but was implicitly deleted because `PhysicalCore` has fields
of reference type. Switch to explicitly deleting it to avoid a Clang
warning.
The move *constructor* is still defaulted, and is required to exist due
to the use of `std::vector<PhysicalCore>`.
|
|
- Add a type check so that calling Push with an invalid type produces a
compile error rather than a linker error.
- vi.cpp was calling Push with a variable of type `std::size_t`.
There's no explicit overload for `size_t`, but there is one for `u64`,
which on most platforms is the same type as `size_t`. On macOS,
however, it isn't: both types are 64 bits, but `size_t` is `unsigned
long` and `u64` is `unsigned long long`. Regardless, it makes more
sense to explicitly use `u64` here instead of `size_t`.
|
|
__unix__ is not predefined on Apple platforms even though they are Unix.
|
|
|
|
Clang complains about `new_chunk`'s constructor using the
then-uninitialized `first_chunk` (even though it's just to get a pointer
into it).
|
|
|
|
|
|
video_core: Adjust `NUM` macro to avoid Clang warning
|
|
already declared in the superclass
|
|
The previous definition was:
#define NUM(field_name) (sizeof(Maxwell3D::Regs::field_name) / sizeof(u32))
In cases where `field_name` happens to refer to an array, Clang thinks
`sizeof(an array value) / sizeof(a type)` is an instance of the idiom
where `sizeof` is used to compute an array length. So it thinks the
type in the denominator ought to be the array element type, and warns if
it isn't, assuming this is a mistake.
In reality, `NUM` is not used to get array lengths at all, so there is no
mistake. Silence the warning by applying Clang's suggested workaround
of parenthesizing the denominator.
|
|
yuzu_cmd: Remove 'users_size'
|
|
maxwell_dma: Rename RenderEnable::Mode::FALSE and TRUE to avoid name conflict
|
|
Port citra-emu/citra#5577: "Update cubeb and request a persistent stream session"
|
|
xbyak_abi: Avoid implicit sign conversions
|
|
game_list_p: Resolve deprecated usage of QVariant operator<
|
|
|
|
- We were reload'ing the old current scheduler, which may have changed.
|
|
|
|
review feedback.
|
|
|
|
review feedback.
|
|
|
|
|
|
|
|
review feedback.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
- This is closer to pre-multicore behavior, and works a bit better.
|
|
|
