Avoids potentially leaving this variable uninitialized based off the
loader failing to retrieve the ID value.

We can change this into an early-return if the filename is empty.
There's no need to include all of the code within the if statement.

This is user-facing text, so it should be marked as translatable by Qt.

Gets rid of truncation warnings about conversion to int. While we're at
it, we can also de-hardcode the buffer size being used.

We can make the enum class type compatible with fmt by providing an
overload of operator<<.

While we're at it, perform proper bounds checking. If something exceeds
the array, it should be a hard fail, because it's, without a doubt, a
programmer error in this case.

Avoids unnecessary rebuilds of control data on every layer of recursion in AddFstEntriesToGameList

Prompts for title type on NCA files.

Gets the class out of the global namespace.

Silences a warning about truncating from size_t to u32

We don't need to use a heap-allocated std::vector here, given we
explicitly know the bounds.

This is called automatically anyways.

clang-format fix

Full list of new errors and descriptions in core/loader/loader.h

Instead, we make a proper registry class and house it within the main
window, then pass it to whatever needs access to the loaded hotkeys.

This way, we avoid a global variable, and don't need to initialize a
std::map instance before the program can do anything.

* GDBStub works with both Unicorn and Dynarmic now

* Tidy up

This was intermixing signed and unsigned values when they could all just
be signed.

We can just use the file interfaces that Qt provides to prevent needing
to convert to std::string.

The codec used by Qt for const char* and std::string don't necessarily
have to be the same depending on locale. Therefore, we should be using
the correct functions to do the conversions.

There's no need to keep these separate from one another.

Given the function accepts a boolean, we don't need to use an if
statement here and repeat ourselves.

In OnMenuRecentFile() we don't need to construct a QFileInfo instance
just to check if a file exists, we can just use the static member
function to do that (which Qt's documentation also notes as quicker than
constructing an instance).

In UpdateUITheme(), we just want to try and open the file and check the
success of that operation. Technically speaking, between the existence
check and the open call, the file can be deleted or moved, but still
appear to succeed in code. i.e.

1. Existence check -> Returns true
2. File is moved/deleted
3. Open is called, the return value of which isn't checked
4. Nonsense behavior

This way we combine the existence check and the open into one.

Previously, when canceling out of the Load Folder dialog, a user would
get an error dialog about the selected folder not containing a main
file, however, by canceling out of the dialog, no selection was actually
made.

These occur automatically without the need to call them. While we're at
it, also std::move the QString instance into its member variable.