| Age | Commit message (Collapse) | Author |
|---|
|
|
|
core/memory: Deglobalize memory management code
|
|
file_sys/directory: Make EntryType an enum class
|
|
This can trivially be an enum class rather than a regular enum, making
it more strongly typed.
|
|
With all of the trivial parts of the memory interface moved over, we can
get right into moving over the bits that are used.
Note that this does require the use of GetInstance from the global
system instance to be used within hle_ipc.cpp and the gdbstub. This is
fine for the time being, as they both already rely on the global system
instance in other functions. These will be removed in a change directed
at both of these respectively.
For now, it's sufficient, as it still accomplishes the goal of
de-globalizing the memory code.
|
|
Amends a few interfaces to be able to handle the migration over to the
new Memory class by passing the class by reference as a function
parameter where necessary.
Notably, within the filesystem services, this eliminates two ReadBlock()
calls by using the helper functions of HLERequestContext to do that for
us.
|
|
* core_timing: Use better reference tracking for EventType.
- Moves ownership of the event to the caller, ensuring we don't fire events for destroyed objects.
- Removes need for unique names - we won't be using this for save states anyways.
|
|
service: Update function tables
|
|
kernel objects. (#3154)
* kernel: Replace usage of boost::intrusive_ptr with std::shared_ptr for kernel objects.
- See https://github.com/citra-emu/citra/pull/4710 for details.
|
|
service/am: Remove unnecessary Skip calls
|
|
am: Stub QueryApplicationPlayStatistics
|
|
core: Make most implicit type conversion warnings errors on MSVC
|
|
We can simplify these by wrapping the necessary members in structs and
then simply reading out the whole struct.
|
|
Maintains implementation parity between QueryApplicationPlayStatistics
and QueryApplicationPlayStatisticsByUid.
These function the same behaviorally underneath the hood, with the only
difference being that one allows specifying a UID.
|
|
Implement stub for IApplicationFunctions::QueryApplicationPlayStatisticsByUid
|
|
Keeps the function tables up to date.
Updated based off information from Switchbrew.
|
|
These functions are marked for deprecation and it's recommended that the
*_ret variants be used instead.
|
|
These are fairly trivial to resolve and most of the changes entail
using RESULT_UNKNOWN over ResultCode(-1).
|
|
|
|
kernel: Improve events
|
|
hid: Stub SetNpadJoyAssignmentModeSingle and GetNpadHandheldActivationMode
|
|
- Zero initialization here is useful for determinism.
|
|
- This does not actually seem to exist in the real kernel - games reset these automatically.
# Conflicts:
# src/core/hle/service/am/applets/applets.cpp
# src/core/hle/service/filesystem/fsp_srv.cpp
|
|
SetApplicationCopyrightImage and SetApplicationCopyrightVisibility
These commands require Screenshots to be implemented anyway, so they are safe to stub for now.
|
|
hid/npad: Minor cleanup
|
|
ConnectAllDisconnectedControllers()
We should be setting the connection state to true, otherwise we aren't
actually making the controllers connected like the function name
indicates.
|
|
While not an issue, it does prevent fallthrough from occurring if
anything is ever added after this case (unlikely to occur, but this
turns a trivial "should not cause issues" into a definite "won't cause
issues).
|
|
Performs the same behavior, but is built into the core language itself.
No functional change.
|
|
The const qualified variant can also be called in non-const contexts, so
we can remove the non-const variant to eliminate a bit of code
duplication.
|
|
Clearly separate these from the variable declarations to make them more
visible.
|
|
over a map
While a map is an OK way to do lookups (and usually recommended in most
cases), this is a map that lives for the entire duration of the program
and only deallocates its contents when the program terminates.
Given the total size of the map is quite small, we can simply use a
std::array of pairs and utilize std::find_if to perform the same
behavior without loss of performance.
This eliminates a static constructor and places the data into the
read-only segment.
While we're at it, we can also handle malformed inputs instead of
directly dereferencing the resulting iterator.
|
|
This doesn't modify instance state, so it can be made const qualified.
|
|
General fixes to Async GPU
|
|
{bcat, gpu, nvflinger}: Remove trivial usages of the global system accessor
|
|
Removes all uses of the global system accessor within the BCAT
interface.
|
|
|
|
|
|
|
|
|
|
|
|
Services::ES fix casting warnings
|
|
lm: Rewrite logger to use core reporting services
|
|
Makes use of the already existing DeactivateController function.
|
|
|
|
core: Remove Core::CurrentProcess()
|
|
We need to perform explicit casts here, otherwise we're implicitly
truncating a 64-bit type to a 32-bit one.
|
|
Without this, the std::move within the constructor initializer list
won't be able to actually perform a move.
|
|
Allows us to remove a constructor initializer list.
|
|
Given the string is appended to another, we can make it a view so a
redundant full copy of the string isn't made.
|
|
This member function doesn't modify internal member state, so it can be
marked const.
|
