Given the offset is assigned a fixed value in the constructor, we can
just assign it directly and get rid of the need to write the name of the
variable again in the constructor initializer list.

Given the disk shader cache contains non-trivial types, we should
default it in the cpp file in order to prevent inlining of the
complex destruction logic.

The standard library expects hash specializations that don't throw
exceptions. Make this explicit in the type to allow selection of better
code paths if possible in implementations.

We don't need to load the code into a vector and then construct a string
over the data. We can just create a string with the necessary size ahead
of time, and read the data directly into it, getting rid of an
unnecessary heap allocation.

std::move does nothing when applied to a const variable. Resources can't
be moved if the object is immutable. With this change, we don't end up
making several unnecessary heap allocations and copies.

Booleans don't have a guaranteed size, but we still want to have them
integrate into the disk cache system without needing to actually use a
different type. We can do this by supplying non-template overloads for
the bool type.

Non-template overloads always have precedence during function
resolution, so this is safe to provide.

This gets rid of the need to smatter ternary conditionals, as well as
the need to use u8 types to store the value in.

yuzu/{about_dialog, main}: Specify string conversions explicitly for SCM-related info

yuzu_cmd: Make OpenGL's context current

yuzu_cmd: Use OpenGL compat when asked in the settings

qt/configure_graphics: Shadow options at runtime

Compatibility profile and the disk shader cache settings shouldn't
be changed at runtime. This aims to address that shadowing those
options.

The SDL2 frontend never bound the OpenGL context, resulting on a white
screen and no-ops all over the backend.

video_core/memory_manager: Minor tidying

video_core: Remove a few unused variables and functions

CMakeLists: Specify /volatile:iso for MSVC

CMakeLists: Explicitly specify -Wall for the non-MSVC case

Rasterizer Cache: refactor flushing & optimize memory usage of surfaces

Corrects the typo in its name and marks the function as a const member
function, given it doesn't actually modify memory manager state.

Prevents implicit converting constructions of the memory manager.

Makes the class less surprising when it comes to forward declaring the
type, and also prevents inlining the destruction code of the class,
given it contains non-trivial types.

Corrects references to non-existent parameters and corrects typos.

These are able to be omitted from the declaration of functions, since
they don't do anything at the type system level. The definitions of the
functions can retain the use of const though, since they make the
variables immutable in the implementation of the function where they're
used.

Silences a -Wreorder warning.

This isn't used at all, so it can be removed.

This variable is unused entirely, so it can be removed.

Instead of retrieving the data from the std::variant instance, we can
just check if the variant contains that type of data.

This is essentially the same behavior, only it returns a bool indicating
whether or not the type in the variant is currently active, instead of
actually retrieving the data.

Silences a few compilation warnings.

By default, MSVC doesn't use standards-compliant volatile semantics.
This makes it behave in a standards-compliant manner, making
expectations more uniform across compilers.

service/audctl: Update documentation comments to be relative to 8.0.0

src/CMakeLists: Add /Zc:externConstexpr to the MSVC build flags

Fix Layered ASTC Textures

gl_shader_disk_cache: Skip stored shader variants instead of asserting

Port citra-emu/citra#4749: "web_service: Misc fixes"

yuzu/hotkeys: Remove unnecessary constructor

yuzu/compatdb: Remove unnecessary qualifiers

configure_dialog: Remove the Whats This? button from the dialog

core/memory: Remove unused FlushMode enum