- Used in Super Mario 3D All-Stars

arm_dynarmic_cp15: Initialize member variables

service/sm: Eliminate dependency on the global system instance

service: Remove unused funcation

command_generator/nfp: Eliminate unnecessary copies

decoder/texture: Eliminate narrowing conversion in GetTldCode()

decode/image: Eliminate switch fallthrough in DecodeImage()

hid/configuration: Implement motion controls to HID

Fortunately this didn't result in any issues, given the block that code
was falling through to would immediately break.

The assignment was previously truncating a u64 value to a bool.

In a lot of cases, we can make use of const references rather than
non-const references.

While we're in the area we can silence some truncation and sign
conversion warnings.

GetAmiiboBuffer() returns by const reference, so we can use a reference
instead of taking the returned buffer by value.

Ensures that the member variables are always initialized to a
deterministic value on creation.

control_metadata: Resolve typo in Portuguese language name

This isn't used anywhere, so this is a trivial fix.

This is now completely unused, so it can be removed.

We can check the end of the string first for null-termination, rather
than the beginning of the string.

gc_adapter: Disable MSVC nonstandard extension warning on libusb.h

video_core: Enforce -Werror=switch

bis_factory/romfs_factory: Eliminate dependencies on the global system instance

core/loader: Remove dependencies on the global system instance

This forces us to fix all -Wswitch warnings in video_core.

Now all that remains is:

18 instances in file_sys code
14 instances in GDB stub code (this can be tossed wholesale)
4 instances in HLE code
2 instances in settings code.

 nca_patch: Reduce stack usage size within SearchBucketEntry()

cheat_engine: Remove unnecessary system argument to CheatParser's Parse function

Previously this function was using ~16KB of stack (16528 bytes), which
was caused by the function arguments being taken by value rather than by
reference.

We can make this significantly lighter on the stack by taking them by
reference.

This is only used internally and doesn't depend on any class state, so
we can make it fully internal.

We don't need to create two separate instantiations of the same code, we
can simply make the character template argument a regular function
parameter.

This isn't used within the function at all in any implementations, so we
can remove it entirely.

patch_manager: Minor cleanup

We make it explicit that we're truncating arithmetic here to resolve
compiler warnings (even if the sizes weren't u32/u64 arithmetic
generally promotes to int :<)

We can use these to avoid typing the same type redundantly. This way, if
these ever change, only a single location needs to be modified.

These functions are only used within this translation unit, so we can
make them internally linked.

Pragma disable zero-sized array nonstandard extension warning on MSVC.

crypto/key_manager: Remove dependency on the global system accessor