This was initially added to prevent problems from stubbed/not implemented NFC services, but as we never encountered such and as it's only used in a deprecated function anyway, I guess we can just remove it to prevent more clutter of the settings.

kernel/process: Remove unused AddressMapping struct

core/core_timing: Make callback parameters consistent

kernel/kernel: Remove unnecessary forward declaration

core/cheat_engine: Make MemoryReadImpl and MemoryWriteImpl internally linked

These don't need to be visible outside of the translation unit, so they
can be enclosed within an anonymous namespace.

In some cases, our callbacks were using s64 as a parameter, and in other
cases, they were using an int, which is inconsistent.

To make all callbacks consistent, we can just use an s64 as the type for
late cycles, given it gets rid of the need to cast internally.

While we're at it, also resolve some signed/unsigned conversions that
were occurring related to the callback registration.

This is no longer necessary, as ResultVal isn't used anywhere in the
header.

Another leftover from citra that's now no longer necessary.

 core/hle/kernel: Split transfer memory handling out into its own class

set_sys: Implement GetFirmwareVersion(2) for libnx hosversion

Migrate off directly modifying CMAKE_* compilation-related flags directly

loader/nso: Minor refactoring

file_sys/cheat_engine: Remove use of global system accessors

gpu: Rewrite MemoryManager based on the VMManager implementation.

Revert "Devirtualize Register/Unregister and use a wrapper instead."

Makes it impossible to indirectly violate the ODR in some other
translation unit due to these existing.

Instead, pass in the core timing instance and make the dependency
explicit in the interface.

Now that the NSO header has the proper size, we can just use sizeof on
it instead of having magic constants.

This source file was utilizing its own version of the NSO header.
Instead of keeping this around, we can have the patch manager also use
the version of the header that we have defined in loader/nso.h

The total struct itself is 0x100 (256) bytes in size, so we should be
providing that amount of data.

Without the data, this can result in omitted data from the final loaded
NSO file.

These correspond to the NSOBuildHeader.

core/hle/kernel: Make Mutex a per-process class.

core/memory: Remove unnecessary includes

- Fixes graphical issues from transitions in Super Mario Odyssey.

kernel/vm_manager: Amend flag value for code data

service/am: Add function table for IDebugFunctions

file_sys: Implement parser and interpreter for game memory cheats

We already have the service related stuff set up for this, however, it's
missing the function table.

Makes it more evident that one is for actual code and one is for actual
data. Mutable and static are less than ideal terms here, because
read-only data is technically not mutable, but we were mapping it with
that label.

This should actually be using the data flags, rather than the code
flags.

In 93da8e0a, the page table construct
was moved to the common library (which utilized these inclusions). Since
the move, nothing requires these headers to be included within the
memory header.

input_common/sdl: Correct return values within GetPollers implementations

Introduced as a result of #2090, we already define the copy constructor
further down below, so this isn't needed.

Port citra-emu/citra#4244 and citra-emu/citra#4599: Changes to BitField

file_sys/content_archive: Amend name of Data_Unknown5 enum entry

 common/uint128: Add missing header guard