common/concepts: Move <type_traits> include out of the Common namespace

Revert "common/time_zone: Simplify GetOsTimeZoneOffset()"

common: Silence two discarded result warnings

vulkan/wrapper: Avoid unnecessary copy in EnumerateInstanceExtensionProperties()

This is a compiler/linker error waiting to happen.

main: Fix Open Save/Mod Locations for installed titles

configuration_shared: Simplify name lookup in highlighting functions

common/fileutil: Convert namespace to Common::FS

common/time_zone: Simplify GetOsTimeZoneOffset()

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.

Migrates a remaining common file over to the Common namespace, making it
consistent with the rest of common files.

This also allows for high-traffic FS related code to alias the
filesystem function namespace as

namespace FS = Common::FS;

for more concise typing.

These are intentionally discarded internally, since the rest of the
public API allows querying success. We want all non-internal uses of
these functions to be explicitly checked, so we can signify that we
intentionally want to discard the return values here.

We can simplify this function down into a single line with the use of
fmt. A benefit with the fmt approach is that the fmt variant of
localtime is thread-safe as well, making GetOsTimeZoneOffset()
thread-safe as well.

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