* Swap "Plus" with "Minus" on the controller GUI

Major fix /s

Namespaces all OpenGL code under the OpenGL namespace.

Prevents polluting the global namespace and allows clear distinction
between other renderers' code in the future.

Used by Splatoon 2's debug menu.

Should fix any compile errors

* Added bfttf loading

We can now load system bfttf fonts from system archives AND shared memory dumps. This allows people who have installed their system nand dumps to yuzu to automatically get shared font support. We also now don't hard code the offsets or the sizes of the shared fonts and it's all calculated for us now.

* Addressed plu fixups

* Style changes for plu

* Fixed logic error for plu and added more error checks.

- This reverts commit 3ef4b3d4.
- This commit had broken a lot of games. We really should do a full implementation of this in one change.

Resolves a -Wlogical-op-parentheses warning.

Avoids the need to rebuild these source files if the mode header
changes.

Previously this would cause warnings about implicit conversions to s16
from a double

The previous form of initializing done here is a C-ism, an empty set of
braces is sufficient for initializing (and doesn't potentially cause
missing brace warnings, given the first member of the struct is a COORD
struct).

Gets rid of the potential for C array-to-pointer decay, and also makes
pointer arithmetic to get the end of the copy range unnecessary. We can
just use std::array's begin() and end() member functions.

25us is far too small, and would result in std::this_thread::sleep_for
being called with this as a maximum value. This means that a guest
application that produces frames instantly would only be limited to
40 kHz.

25ms is a more appropriate value, as it allows for a 60 Hz refresh
rate while providing enough slack in the negative region.

This doesn't require the central core header to be included, it just
needs the vfs headers.

Gets rid of a TODO that is long overdue.

LOG_TRACE is only enabled on debug builds which can be quite slow when
trying to debug graphics issues. Instead we can log the messages to the
debug log, which is available on both release and debug builds.

Avoids the need to rebuild multiple source files if the filesystem code
headers change.

This also gets rid of a few instances of indirect inclusions being
relied upon

Prevents potential compilation errors from occuring due to multiple
inclusions

We'll write <last color output reg + 2> to gl_FragDepth.

The X and Y coordinates should be in gpr8 and gpr8+1, respectively.

This fixes the cutscene rendering in Sonic Mania.

Avoids the need to rebuild whatever includes the romfs factory header if
the loader header ever changes. We also don't need to include the main
core header. We can instead include the headers we specifically need.

telemetry_session: Don't allocate std::string instances for program lifetime in GetTelemetryId() and RegenerateTelemetryId()

Given these functions aren't intended to be used frequently, there's no
need to keep the std::string instances allocated for the whole lifetime
of the program. It's just a waste of memory.

This is only used in a shared_ptr, so we can forward declare it.

We have an overload of WriteBuffer that accepts containers that satisfy
the ContiguousContainer concept, which std::array does, so we only need
to pass in the array itself.

Arranges them in the order the members would be initialized

This is no longer used, so it can be removed.

Makes it a little bit more self-documenting.

Given the source and destination types are the same std::array type, we
can simply use regular assignment to perform the same behavior.

Avoids the need to repeatedly specify the whole array type in multiple
places.

ProfileInfo is quite a large struct in terms of data, and we don't need
to perform a copy in these instances, so we can just pass constant
references instead.

This doesn't modify the passed in array, so this can be a const
reference.

This can just be constexpr like the others

We can use the constructor initializer list and just compare the
contained u128's together instead of comparing each element
individually. Ditto for comparing against an invalid UUID.