applet_resource_user_id isn't actually modified and is just assigned to
a member variable, so this doesn't need to be a mutable reference.

Similarly, the device name itself isn't modified and is only moved. We
pass by value here, since we can still perform the move, but eliminate a
sneaky set of calls that can unintentionally destroy the original
string. Given how nested the calls are, it's good to get rid of this
potential vector for a use-after-move bug.

concepts: Use the std::contiguous_iterator concept

This also covers std::span, which does not have a const iterator.

Also renames IsSTLContainer to IsContiguousContainer to explicitly convey its semantics.

tests: fix for -Wall

service: nfp: Allow amiibos without keys

input_common: cache vibration tests

Fix #9123

video_core: Reimplementing the maxwell drawing trigger mechanism

core: barrier service thread shutdown

video_core: Fix legacy to generic location unpaired

nvdrv: fix container destruction order

general: Treat more warnings as errors

Clang erroneously emits this warning when using designated initializers.

Copies up to sizeof(p_name) - 1 in strncpy and null terminates it at p_name[254]

Clang and ICC for whatever reason also defines __GNUC__. Exclude them from this check.

This is similar to Clang's -Wunused-lambda-capture

Disabling C4100 is similar to -Wno-unused-parameter

These warnings are already included in /W3.

Resolves C4146 on MSVC

These are already explicitly or implicitly set in src/CMakeLists.txt

Maxwell3D/Puller: Fix regressions and syncing issues.

hid/npad: Fix copy size in GetSupportedNpadIdTypes

format_lookup_table: Implement R32_B24G8 with D32_FLOAT_S8_UINT

k_session_request: Add missing override