Kirby input still doesn't work, should fix a lot of other games

Used by Mario Party

HandheldVariant is for specific games which expect handheld controllers to be at position 8(kirby), however this doesn't fix all games as some games require handhelds to be at position 0(snipperclips)

Fixes arms

gl_shader_decompiler: Remove unused variables in TMML's implementation

Port citra-emu/citra#4310: "Handle touch input"

svcBreak, Signalling to the debugger should not kill execution

telemetry_session: Minor miscellaneous changes

ips_layer: Minor miscellaneous changes

Fixed smo softlock due to incorrect effect state updating

Given "y" isn't always used, but "x" is, we can rearrange this to avoid
unused variable warnings by changing the names of op_a and op_b

There's no "func" parameter, so this can just be removed.

Prevents potential compilation issues in the future by including missing
headers for certain functions and types.

This isn't implemented anywhere, so it can just be removed.

We don't need to potentially heap-allocate a std::string instance here,
given the data is known ahead of time. We can just place it within an
array and pass this to the mbedtls functions.

change TouchToPixelPos to return std::pair<int, int>

static_cast (SDL)

various minor style and code improvements

style - PascalCase for function names

made touch events private

const pointer arg in touch events

make TouchToPixelPos a const member function

did I do this right?

braces on barely-multiline if

remove question comment (confirmed in Discord)

fixed consts

remove unused parameter from TouchEndEvent

DRY - High-DPI scaled touch put in separate function

also fixes a bug where if you start touching (with either mouse or touchscreen) and drag the mouse to the LEFT of the emulator window, the touch point jumps to the RIGHT side of the touchscreen; draggin to above the window would make it jump to the bottom.

implicit conversion from QPoint to QPointF, apparently

I have no idea what const even means but I'll put it here anyway

remove unused or used-once variables

make touch scaling functions const, and put their implementations together

removed unused FingerID parameters

QTouchEvent forward declaration; add comment to TouchBegin that was lost in an edit

better DRY in SDL

To do -> TODO(NeatNit)

remove unused include

We can just compare the existing std::vector instance with a constexpr
std::array containing the desired match. This is lighter resource-wise,
as we don't need to allocate on the heap.

Makes the layout of the array consistent, by making all elements match,
instead of special-casing the first one.

Adds missing includes to prevent potential compilation issues in the
future. Also moves the definition of a struct into the cpp file, so that
some includes don't need to be introduced within the header.

We don't need to make a copy of the read data, so we can std::move it
into the make_shared call here.

When loading NROs, svcBreak is called to signal to the debugger that a new "module" is loaded. As no debugger is technically attached we shouldn't be killing the programs execution.

 fsmitm_romfsbuild: Add support for stubbing and IPS patches in LFS

ips_layer: Add support for IPSwitch executable patches

Fixed assertion due to CountAddOnContent & Casting warnings

Unmapping an unmapped buffer should succeed

nso/nro: Add support for passing command-line arguments to executable

Hardware tests show that trying to unmap an unmapped buffer already should always succeed. Hardware test was tested up to 32 iterations of attempting to unmap

Uses -p (--program) and following string as args.

Only added if arguments string is non-empty and a pass is requested by loader.