Revert "common/time_zone: Simplify GetOsTimeZoneOffset()"

common: Silence two discarded result warnings

vulkan/wrapper: Avoid unnecessary copy in EnumerateInstanceExtensionProperties()

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

textures/decoders: Fix block linear to pitch copies

We can query the given object name directly from the widget itself. This
removes any potential for forgetting to change the name if the widget
gets renamed and makes the API much simpler (just pass in the widget,
and not worry about its name).