service/am: Add missing return in error case for IStorageAccessor's Read/Write()

gl_rasterizer: Pass the right number of array quad vertices count

gl_rasterizer: Limit OpenGL point size to a minimum of 1

 video_core/engines/engine_upload: Minor tidying

maxwell_to_gl: Add TriangleFan primitive topology

ipc_helpers: Amend floating-point type in Pop<double> specialization

 yuzu/debugger: Specify string conversions explicitly 

core/kernel/object: Rename ResetType enum members for clarity

kernel/svc: Mark GetThreadList() and UnmapProcessCodeMemory() as internally linked

yuzu/applets/profile_select: Mark header string as translatable

gl_shader_gen: std::move objects where applicable

CMakeLists: Handle VS 2019 in a less annoying manner

yuzu/bootmanager: Explicitly enable deprecated OpenGL features on compat

yuzu/util: Remove unused spinbox.cpp/.h

This has been left unused since the removal of the vestigial surface
viewer. Given it has no uses left, this can be removed as well.

Avoids performing copies into the pair being returned. Instead, we can
just move the resources into the pair, avoiding the need to make copies
of both the std::string and ShaderEntries struct.

This is a user-facing string, so it should be marked as translatable.

Allows the graphics breakpoints to compile with implicit string
conversions disabled.

This allows the microprofile widget to compile with implicit string
conversions disabled.

Allows compiling the wait tree widget with implicit string conversions
disabled.

Currently, this overload isn't used, so this wasn't actually hit in any
code, only the float overload is used.

These are only used from within this translation unit, so they don't
need to have external linkage. They were intended to be marked with this
anyways to be consistent with the other service functions.

Previously this would fall through and return successfully, despite
being an out of bounds read or write.

Renames the members to more accurately indicate what they signify.
"OneShot" and "Sticky" are kind of ambiguous identifiers for the reset
types, and can be kind of misleading. Automatic and Manual communicate
the kind of reset type in a clearer manner. Either the event is
automatically reset, or it isn't and must be manually cleared.

The "OneShot" and "Sticky" terminology is just a hold-over from Citra
where the kernel had a third type of event reset type known as "Pulse".
Given the Switch kernel only has two forms of event reset types, we
don't need to keep the old terminology around anymore.

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.

Nvidia's proprietary driver creates a real OpenGL compatibility profile
without this option, meanwhile Intel (and probably AMD, I haven't tested
it) require that QSurfaceFormat::FormatOption::DeprecatedFunctions is
explicitly enabled.

VS 2019 is binary compatible with VS 2017, so we can safely use
the prebuilt libraries for VS 2017 with VS 2019. This makes it less
annoying to build yuzu with the most up to date toolchain.

Silences a -Wreorder warning.

Avoids inlining destruction logic where applicable, and also makes
forward declarations not cause unexpected compilation errors depending
on where the State class is used.

These only apply in the definition of the function. They can be omitted
from the declaration.