The kernel uses the handle table of the current process to retrieve the
process that should be used to retrieve certain information. To someone
not familiar with the kernel, this might raise the question of "Ok,
sounds nice, but doesn't this make it impossible to retrieve information
about the current process?".

No, it doesn't, because HandleTable instances in the kernel have the
notion of a "pseudo-handle", where certain values allow the kernel to
lookup objects outside of a given handle table. Currently, there's only
a pseudo-handle for the current process (0xFFFF8001) and a pseudo-handle
for the current thread (0xFFFF8000), so to retrieve the current process,
one would just pass 0xFFFF8001 into svcGetInfo.

The lookup itself in the handle table would be something like:

template <typename T>
T* Lookup(Handle handle) {
    if (handle == PSEUDO_HANDLE_CURRENT_PROCESS) {
        return CurrentProcess();
    }

    if (handle == PSUEDO_HANDLE_CURRENT_THREAD) {
        return CurrentThread();
    }

    return static_cast<T*>(&objects[handle]);
}

which, as is shown, allows accessing the current process or current
thread, even if those two objects aren't actually within the HandleTable
instance.

Our implementation of svcGetInfo was slightly incorrect in that we
weren't doing proper error checking everywhere. Instead, reorganize it
to be similar to how the kernel seems to do it.

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).