This was assigning the field to itself, which is a no-op. The size
doesn't change between its initial assignment and this one, so this is a
safe change to make.

cheat_engine: Resolve implicit bool->u64 conversion

gl_shader_cache: Use std::max() for determining num_workers

General: Tidy up clang-format warnings part 2

Does not allocate more threads than available in the host system for boot-time shader compilation and always allocates at least 1 thread if hardware_concurrency() returns 0.

freezer: Make use of std::erase_if

common/virtual_buffer: drop unused includes

General: Tidy up clang-format warnings

cheat_engine: Make use of designated initializers

input_configuration: Add range logic for analog sticks

This PR is now only the Analog devices handling the range value defaulting at 100%

kernel: Remove unused variables

vfs_vector: Make creation of array vfs files less verbose

game_list_worker: Do not clear entries when > 1 gamedir is present

am: Unstub SetScreenShotPermission

video_core: Remove redundant pixel format type

ci: Make use of clang-format 10.0

common/concepts: Rename IsBaseOf to DerivedFrom

This makes it more inline with its currently unavailable standardized
analogue std::derived_from.

While we're at it, we can also make the template match the requirements
of the standardized variant as well.

partition_data_manager: Make data arrays constexpr

address_arbiter/scheduler: Resolve sign conversion warnings

Cleans up the callsites in other functions.

VAddr will always be 64-bit, so there's no need to take a trivial
primitive alias by reference.

With C++20 we can simplify the erasing idiom.

We can just return zero here.

Same behavior, but makes the member being assigned obvious.

Fills in some hashes that were previously unhandled.

Previously the constructor for all of these would run at program
startup, consuming time before the application can enter main().

This is also particularly dangerous, given the logging system wouldn't
have been initialized properly yet, yet the program would use the logs
to signify an error.

To rectify this, we can replace the literals with constexpr functions
that perform the conversion at compile-time, completely eliminating the
runtime cost of initializing these arrays.

ipc_helpers: Only allow trivially copyable objects with PushRaw() and PopRaw()

aes_util: Allow SetIV() to be non-allocating

logging/backend: Make use of designated initializers

On DragonFly and NetBSD build fails with

src/common/virtual_buffer.cpp
src/common/virtual_buffer.cpp:16:10: fatal error: sys/sysinfo.h: No such file or directory
 #include <sys/sysinfo.h>
          ^~~~~~~~~~~~~~~

buffer_queue: Make use of designated initializers/std::nullopt where applicable

CMakeLists: Resolve #4478