While DEPBAR is stubbed it doesn't change anything from our end. Shading
languages handle what this instruction does implicitly. We are not
getting anything out fo this log except noise.

 texture_cache: Use a flat table instead of switch for texture format lookups

common/logging: Silence no return value warnings

Revert "common/bit_field: Silence sign-conversion warnings"

common/bit_field: Silence sign-conversion warnings

gl_rasterizer: Emulate viewport flipping with ARB_clip_control

core: Make most implicit type conversion warnings errors on MSVC

We can just use numeric_limits instead of relying on wraparound behavior
here.

format_lookup_table: Drop bitfields

format_lookup_table: Use std::array for definition table

format_lookup_table: Include <limits> instead of <numeric>

Use a large flat array to look up texture formats. This allows us to
properly implement formats with different component types. It should
also be faster.

common_funcs: Remove semicolons from INSERT_PADDING_* macros

Makes code that uses the macros consistent by requiring the lines to be
terminated with a semicolon.

Abstracted ComponentType was not being used in a meaningful way.
This commit drops its usage.

There is one place where it was being used to test compatibility between
two cached surfaces, but this one is implied in the pixel format.
Removing the component type test doesn't change the behaviour.

Complete the implementation of RGBA16UI

Implement stub for IApplicationFunctions::QueryApplicationPlayStatisticsByUid

core: Migrate off deprecated mbedtls functions

key_manager: Make use of IOFile in WriteKeyToFile()

shader: Implement FSWZADD and reimplement SHFL

common/hash: Remove unused HashableStruct

xts_archive: Remove redundant std::string constructor

This is unused, so it can be removed. There's better ways of ensuring
zeroed out padding bits, like using zero-initialization, anyhow.

common_funcs: silence sign-conversion warnings in MakeMagic()

We can just call the .data() member of path instead of constructing a
completely new string.

We can trivially resolve these by casting the characters to unsigned
values and then shifting the bits.

video_core: Treat implicit conversions as errors

Move source in windows uploads into its own internal zip

This should prevent path length issues when extracting the build from the installer

ci: Package source with builds

This properly handles unicode-based paths on Windows, while opening a
raw stream doesn't out-of-the-box.

Prevents file creation from potentially failing on Windows PCs that make
use of unicode characters in their save paths (e.g. writing to a user's
AppData folder, where the user has a name with non-ASCII characters).

These functions are marked for deprecation and it's recommended that the
*_ret variants be used instead.

Since the introduction of this library, numerous improvements have been
made. Notably, many of the warnings we would get by simply including the
library header have now been fixed. This makes it much easier to make
conversion warning an error.

These are fairly trivial to resolve and most of the changes entail
using RESULT_UNKNOWN over ResultCode(-1).

We simply need to turn the literal argument to std::accumulate into a
double, rather than an int.

Uncovered a bug within Thread's SetCoreAndAffinityMask() where an
unsigned variable (ideal_core) was being compared against "< 0", which
would always be a false condition.

We can also get rid of an unused function (GetNextProcessorId) which contained a sign
mismatch warning.