Rather than keeping around unused values, we can just introduce them as
needed.

There's no real point to keeping the separate enum around, especially
given the name of the error code itself is supposed to document what the
value actually represents.

Keeps filesystem-related error codes in one spot.

* Fixed priority switching edge case for handheld

We accidently used controller index instead of npad id

* Moved NPadIdToIndex

csrng: Use random integer distribution instead of raw engine

Prevents returning the same value every single call.

kernel/thread: Deduplicate scheduler switching code

patch_manager: Add support for dumping uncompressed NSOs

service/audren_u: Forward RequestUpdateAuto through the same function as RequestUpdate

Improved GPU Caches lookup Speed

acc/profile_manager: Minor cleanup-related changes

service/acc: Correct error case within TrySelectUserWithoutInteraction()

yuzu/configure_system: Mark the entropy mask string as nontranslatable

empty() in this case will always return false, since the returned
container is a std::array. Instead, check if all given users are invalid
before returning the error code.

Avoids relying on fmt always being indirectly included.

nfp: Correct erroneous sizeof expression within GetTagInfo()

hid/npad: Add missing break in switch statement within Controller_NPad::OnUpdate

service: Mark MakeFunctionString with the [[maybe_unused]] attribute.

There's no need for translators to concern themselves with the
validation mask used by the entry field.

common/string_util: Minor cleanup

svc: Use proper random entropy generation algorithm

The previous expression would copy sizeof(size_t) amount of bytes (8 on
a 64-bit platform) rather than the full 10 bytes comprising the uuid
member.

Given the source and destination types are the same, we can just use an
assignment here instead.

Based off RE, they both currently go through the same codepath with no
difference in behavior.

When yuzu is compiled in release mode this function is unused, however,
when compiled in debug mode, it's used within a LOG_TRACE statement.
This prevents erroneous compilation warnings about an unused function
(that isn't actually totally unused).

The code in both places was the same verbatim, so we can extract it to a
function to deduplicate the logic.

gl_rasterizer_cache: CopySurface optimization

qt: Move Open yuzu Folder action from Help to File

video_core/renderer_base: Remove GL include from the renderer base class files

engines/maxwell_3d: Minor cleanup

 kernel/process: Migrate heap-related memory management out of the process class and into the vm manager

hle/audren_u: Implement Get/SetRenderingTimeLimit

audio_core/audio_renderer: Fix typo in AuxInfo member name

[ns|fsp_srv]: Implement various functions to boot Checkpoint

An old function from Dolphin. This is also unused, and pretty inflexible
when it comes to printing out different data types (for example, one
might not want to print out an array of u8s but a different type
instead. Given we use fmt, there's no need to keep this implementation
of the function around.

This is an unused hold-over from Dolphin that was primarily used to
parse values out of the .ini files. Given we already have libraries that
do this for us, we don't need to keep this around.

Keeps the base class source files implementation-agnostic.

This is currently unused and doesn't really provide much value to keep
around either.