| Age | Commit message (Collapse) | Author |
|
network, sockets: Replace `POLL_IN`, `POLL_OUT`, etc. constants with an `enum class PollEvents`
|
|
core: Remove unnecessary enum casts in log calls
|
|
applets: Resolve variable shadowing
|
|
Follows the video core PR. fmt doesn't require casts for enum classes
anymore, so we can remove quite a few casts.
|
|
Avoids unnecessary copies.
|
|
Avoids unnecessary 1072 byte copies when querying info about
controllers.
|
|
core: Mark unused fields as [[maybe_unused]]
|
|
hle: Type check ResponseBuilder::Push arguments, and fix use in vi.cpp
|
|
nvdrv: Remove useless re-declaration of pure virtual methods that were already declared in the superclass
|
|
boxcat: Avoid unnecessary object copy
|
|
video_core: Resolve more variable shadowing scenarios pt.3
|
|
`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.
|
|
`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`.
|
|
|
|
|
|
already declared in the superclass
|
|
Cleans out the rest of the occurrences of variable shadowing and makes
any further occurrences of shadowing compiler errors.
|
|
|
|
Kernel: Refactor to use 4-instances of Dynarmic & various cleanups and improvements
|
|
audio_core: Make shadowing and unused parameters errors
|
|
Moves the audio code closer to enabling warnings as errors in general.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
hid: Check if applet_resource exists in InitializeVibrationDevice
|
|
|
|
HID: Stub set and get NpadCommunicationMode
|
|
Now there's only two meaningful instances left in core.
|
|
|
|
Completely removes all usages of the global system instance within the
services code by passing in the using system instance to the services.
|
|
nvdrv, video_core: Don't index out of bounds when given invalid syncpoint ID
|
|
nvservices: Reintroducee IoctlCtrl
|
|
|
|
- This is used by Super Mario 3D All-Stars.
|
|
* hle: services: Fix a crash with improper NVFlinger lifetime management.
- This crash would happen when attempting to shutdown yuzu early on in boot.
|
|
- Use .at() instead of raw indexing when dealing with untrusted indices.
- For the special case of WaitFence with syncpoint id UINT32_MAX,
instead of crashing, log an error and ignore. This is what I get when
running Super Mario Maker 2.
|
|
svc: Remove unnecessary [[maybe_unused]] tag
|
|
Fixes regression caused by #4907 which caused games like Breath of the Wild 1.0.0 not to boot.
|
|
The parameter is used in this function, so this suppression isn't
necessary.
|
|
patch_manager: Remove usages of the global system instance
|
|
core: Make nvservices more standardized
|
|
|
|
functions.
- Used by Animal Cross: New Horizons v1.6.0 update, minimal stub gets this update working.
|