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.

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

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 

vk_command_pool: Add missing header guard

audio_core: Remove unnecessary inclusions

behavior_info: Fix typo Renerer -> Renderer

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

Add automap feature for GC adapter

Using statements already make these unnecessary.

memory: Resolve a -Wdocumentation warning

install_dialog: Remove unused function prototype

game_list: Eliminate redundant argument copies 

Same behavior, less repetition.

memory doesn't exist as a parameter any more.

Allows the compiler to warn against cases where the return value isn't
used (which would be a bug).

This function doesn't have an implementation, so it can be removed to
prevent others from unintentionally using it.

Makes the naming consistent with the rest of the functions that are
present.

Several functions can be taken by const reference to avoid copies

control_flow: Make use of std::move in InsertBranch()