Corrects references to non-existent parameters and corrects typos.

These are able to be omitted from the declaration of functions, since
they don't do anything at the type system level. The definitions of the
functions can retain the use of const though, since they make the
variables immutable in the implementation of the function where they're
used.

The behavior of the Hotkey constructor is already accomplished via in-class member
initializers, so the constructor is superfluous here.

Keeps the code consistent in regards to how the buttons are referred to.

The C++ standard allows constexpr variables declared with the extern
keyword to have external linkage. Previously MSVC wasn't abiding by
this. This just makes the compiler more standards compliant during
builds.

Given we currently don't make use of anything that would break by this,
this is safe to enable.

Makes it much nicer to visually scan the options. This also starts the
flag descriptions from the same column for the same reason.

Recent changes to memory-related code resulted in this being unused, so
we can remove it.

The backend is not used until we decide to submit the testcase/telemetry, and creating it early prevents users from updating the credentials properly while the games are running.

This isn't used anywhere, so we can get rid of it.

Makes use of src, so it's not considered unused.

Introduced in 72477731. This is only
necessary within header files.

This class is used in a polymorphic context, so destruction of the
context will lead to undefined behavior if the destructor isn't virtual.

Instead of asserting on already stored shader variants, silently skip them.
This shouldn't be happening but when a shader is invalidated and it is
not stored in the shader cache, this assert would hit and save that
shader anyways when the asserts are disabled.

By adding the missing layer offset in ASTC compression.

Unintentionally introduced in 552d5071

The state of these service calls are still the same in version 8.0.0.

This option allows picking the compatibility profile since a lot of bugs
are fixed in it. We devs will use this option to easierly debug current
problems in our Core implementation.:wq

It was possible for "samplers" to be read without being written. This
addresses that.

This call was added to the SVC handlers in the 8.0.0 kernel, so we can
finally give it a name.

These two service functions are literally hardcoded to always return
these values without any other error checking.