configure_hotkeys: Don't translate empty strings

aes_util: Make use of non-template variant of Transcode

yuzu: Make use of qOverload where applicable

vulkan_renderer: Async shader/graphics pipeline compilation

main: Add an option to modify the currrent game's configuration

async_shaders: Resolve -Wpessimizing-move warning

Addressing feedback from Rodrigo

yuzu: Resolve -Wextra-semi warnings

Co-authored-by: Morph <39850852+Morph1984@users.noreply.github.com>

There's no need to translate an empty string. This just gives
translators unnecessary work.

While we're in the same area, we can ensure GameDir member variables are
always initialized to consistent values.

common: Make use of [[nodiscard]] where applicable

maxwell_3d: Resolve -Wextra-semi warning

Creates a new entry in the Emulation menu called "Configure Current Game..." that is only available if a game is currently being executed in yuzu. When selected, it opens the game properties dialog for the current game.

Thanks to @BSoDGamingYT for reminding me to do this.

Seems like all compilers don't support std::span yet.

Now that clang-format makes [[nodiscard]] attributes format sensibly, we
can apply them to several functions within the common library to allow
the compiler to complain about any misuses of the functions.

core: Resolve several -Wextra-semi warnings

software_keyboard: Resolve a pessimizing move warning

system_control: Make functions internally linked where applicable

kernel/scheduler: Mark SchedulerLock constructor as nodiscard 

lz4_compression/zstd_compression: Make use of std::span in interfaces

CMakeLists: Update fmt to 7.0.3

textures/decoders: Fix block linear to pitch copies

Eliminates a verbose function cast.

time_zone_content_manager: Collapse auto and default case

Prevents a useless self-assignment from occurring.

Keeps the library up to date and fixes a few bugs

A std::vector created in place like this is already an rvalue and
doesn't need to be moved.

We can amend one of the cascade macros to require semicolons in order to
compile. In other cases, we can just remove the superfluous semicolons.

Prevents pessimization of the move constructor (which thankfully didn't
actually happen in practice here, given std::thread isn't copyable).

Semicolons after a function definition aren't necessary.

Ensures that callers make use of the constructor, preventing bugs from
silently occurring.

Allows the compiler to warn about cases where the constructor is used
but then immediately discarded, which is a potential cause of
locking/unlocking bugs.

cheat_engine: Resolve implicit bool->u64 conversion

gl_shader_cache: Use std::max() for determining num_workers