We can make this message more meaningful by indicating the location the
screenshot has been saved to. We can also log out whenever a screenshot
could not be saved (e.g. due to filesystem permissions or some other
reason).

Treating it as a u16 can result in a sign-conversion warning when
performing arithmetic with it, as u16 promotes to an int when aritmetic
is performed on it, not unsigned int.

This also makes the interface more uniform, as the layout interface now
operates on u32 across the board.

This class contains non-trivial members, so we should default the
destructor's definition within the cpp file.

Same thing (for platforms we support), less reading.

This function is defined as taking an int, not a bool.

We can just pass a pointer to GMainWindow directly and make it a
requirement of the interface. This makes the interface a little safer,
since this would technically otherwise allow any random QWidget to be
the parent of a render window, downcasting it to GMainWindow (which is
undefined behavior).

We can just invoke these functions by qualifying the object name before
the function.

Makes the conversions explicit, as opposed to implicit.

Allows for things such as:

auto rect = Common::Rectangle{0, 0, 0, 0};

as opposed to being required to explicitly write out the underlying
type, such as:

auto rect = Common::Rectangle<int>{0, 0, 0, 0};

The only requirement for the deduction is that all constructor arguments
be the same type.

Stays consistent in our code with using Qt's provided mechanisms, and
also properly handles Unicode paths (which file streams on Windows don't
do very well).

This is equivalent to specifying two separate functions, so we can just
do that.

Fixes #2522.

The intention behind this commit is to hint someone inspecting an
apitrace dump to ignore this ill-formed GLSL code.

This addresses a bug on geometry shaders where code was being written
before all #extension declarations were done. Ref to #2523

Preserves list of add ons and the icon, which are the two costliest parts of game list population.

There's no performance improvement in passing an unsigned pair by
reference.

We can just make the conversion explicit instead of implicit here to
silence -Wsign-compare warnings.

This was previously performing a size_t == int comparison. Silences a
-Wsign-compare warning.

Nothing from the hotkeys dialog relies on this call occurring, and is
already called from the dialog that calls applyConfiguration().

Avoids mentioning the user and formalizes the error itself.

critical() is intended for critical/fatal errors that threaten the
overall stability of an application. A user entering a conflicting key
sequence is neither of those.

We don't need to extract the entire set of hotkeys into a list and then
iterate through it. We can traverse the list and early-exit if we're
able to.

1. This is something that should be solely emitted by the hotkey dialog
itself
2. This is functionally unused, given there's nothing listening for the
signal.

The previous code was all "smushed" together wasn't really grouped
together that well.

This spaces things out and separates them by relation to one another,
making it easier to visually parse the individual sections of code that
make up the constructor.

QDialogButtonBoxes are horizontal by default.

Uses a std::string_view instead of a std::string, given the pointed to
string isn't modified and is only used in a formatting operation.

This is nice because a few usages directly supply a string literal to
the function, allowing these usages to otherwise not heap allocate,
unlike the std::string overloads.

While we're at it, we can combine the address formatting into a single
formatting call.

Now that all of our code is compilable with implicit QString
conversions, we can enforce it at compile-time by disabling them.

This isn't actually used anywhere, so it can be removed.

Given the std::optional can contain an object type that heap allocates,
we can use std::move to avoid an unnecessary copy/allocation from
occurring.

Qt uses a signed value to represent container sizes, so this was causing
a sign mismatch warning.

Allows the software keyboard applet code to compile with implicit string
conversions disabled.

Allows the error applet to build successfully with implicit string
conversions disabled.

While we're at it, also invert the conditional into a guard clause.