Avoids duplicating the same code twice verbatim.

We don't need to call out to our own file handling functions when we're
going to construct a QFileInfo instance right after it. We also don't
need to convert to a std::string again just to compare the file
extension.

A non-existent parameter was left in some formatting calls (the logging
macro for which only does anything meaningful on debug builds)

 service/set: Implement MakeLanguageCode

This code was around prior to the introduction of PushEnum, so convert
it over so we don't need to cast here.

This function simply converts a given index into a language code.

Uncheck automatically joycons docked when docked mode is enable

Changed logging to be "Log before execution", Added more error logging, all services/svc should now log on some level

Fix clip distance and viewport

Add Depth Clamp Support

npad: Use NPadIdToIndex to prevent invalid array access

gpu: Rewrite GPU command list processing with DmaPusher class.

Texture decoder: Implemented Tile Width Spacing

file_sys/registered_cache: Use regular const references instead of std::shared_ptr for InstallEntry()

yuzu/input: Minor changes

npad: Fix copy/paste error with LED position assignments

Prevents compiler warnings related to truncation when invoking the
dialog. It's also extremely suspect to use a u8 value here instead of a
more general type to begin with.

This doesn't depend on any part of the private interface, so it can be
made a non-member internal function.

profile_manager: Save and load ProfileData from disk

file_sys/registered_cache: Use regular const references instead of std::shared_ptr for InstallEntry()

These parameters don't need to utilize a shared lifecycle directly in
the interface. Instead, the caller should provide a regular reference
for the function to use. This also allows the type system to flag
attempts to pass nullptr and makes it more generic, since it can now be
used in contexts where a shared_ptr isn't being used (in other words, we
don't constrain the usage of the interface to a particular mode of
memory management).

control_metadata: Correct typo in language name (Portugese -> Portuguese)

gl_shader_decompiler: Fixup clip distance index

While we're at it, organize the array linearly, since clang formats the
array elements quite wide length-wise with the addition of the missing
'u'.

Technically also fixes patch lookup and icon lookup with Portuguese,
though I doubt anyone has actually run into this issue.

gl_rasterizer: Fixup for #1723.

std::bind is the pre-C++11 way of doing this.

Orders the elements the way they would actually be initialized in.
Resolves compiler warnings with gcc and clang

The common pattern is to put the data members after the function
interface where applicable.

On invalidating the streaming buffer, we need to reupload all vertex buffers.
But we don't need to reconfigure the vertex format.
This was a (silly) misstake in #1723.

Thanks at Rodrigo for discovering the issue.

Fun fact, as configuring the vertex format also invalidate the vertex buffer,
this misstake had no affect on the behavior.

morton: Fixup compiler warning

gdbstub: Silence value truncation warning within FpuWrite()

- More accurate impl., fixes Undertale (among other games).