frontend/controller: Eliminate dependency on the global system instance

qt/game_list: Give GameListSearchField::KeyReleaseEater a parent

common/wall_clock: Add virtual destructors

Remove ext_extended_dynamic_state blacklist

HID: Use different timing for motion

Latest AMD 20.9.2 driver fixed this, there's no reason to keep it blocked, as the previous stable signed driver release doesn't include the extension.

From -fsanitize=address, this code wasn't calling the proper destructor.
Adding virtual destructors for each inherited class and the base class
fixes this bug.

While we are at it, mark the functions as final.

This is currently necessary to build ffmpeg

main: Allow applets to display on top while fullscreen

codec: Make lookup table static constexpr

cubeb_sink: Use static_cast instead of reinterpret_cast in DataCallback()

nvdrv: Stub nvdec/vic ioctls to bypass nvdec movies

This fixes a memory leak as KeyReleaseEater's destructor was never
called.

audio_core: Resolve sign conversion warnings 

vk_stream_buffer: Fix initializing Vulkan with NVIDIA on Linux

shader/registry: Make use of designated initializers where applicable 

Using the Qt::WindowStaysOnTopHint flag allows these dialogs to show up on top while running in fullscreen. However, if yuzu goes out of focus (by alt-tabbing or otherwise), this flag does not seem to have an effect.

vk_command_pool: Add missing header guard

audio_core: Remove unnecessary inclusions

behavior_info: Fix typo Renerer -> Renderer

This way it's obvious that this function shouldn't be used in any future
code.

Allows compilers to elide needing to push these values on the stack
every time the function is called.

The previous fix only partially solved the issue, as only certain GPUs that needed 9 or less MiB subtracted would work (i.e. GTX 980 Ti, GT 730). This takes from DXVK's example to divide `heap_size` by 2 to determine `allocable_size`. Additionally tested on my Quadro K4200, which previously required setting it to 12 to boot.

Conversions from void* to the proper data type are well-defined and
supported by static_cast. We don't need to use reinterpret_cast here.

Allows compilers to elide needing to push these values on the stack
every time the function is called.

Same behavior, but removes header dependencies where they don't need to
be.

While were at it, we can also enable sign conversion warnings and other
common warnings as errors to prevent these from creeping back into the
codebase.

While we're at it we can make the destructor of the base class virtual
to ensure that any polymorphism issues never occur.

Allows the implementation details to be changed without recompiling any
files that include this header.

service: Restore "unused" function

Turns out this function is actually used, but within a trace log.

arithmetic_integer_immediate: Make use of std::move where applicable

acc: Partially implement LoadOpenContext

Same behavior, minus any redundant atomic reference count increments and
decrements.

renderer_vulkan: Make unconditional use of VK_KHR_timeline_semaphore