Added instead of using a seperate PR to prevent conflicts

Needed by megaman 11

* svcBreak now dumps information from the debug buffer passed

info1 and info2 seem to somtimes hold an address to a buffer, this is usually 4 bytes or the size of the int and contains an error code. There's other circumstances where it can be something different so we hexdump these to examine them at a later date.

* Addressed comments

Fixes input for megaman

Needed for Shantae - Half-Genie Hero - Ultimate Edition!

This should fix crashes when launching multiple games in yuzu

Previously, we would let a user enter an unbounded name and then
silently truncate away characters that went over the 32-character limit.
This is kind of bad from the UX point of view, because we're essentially
not doing what the user intended in certain scenarios.

Instead, we clamp it to 32 characters and make that visually apparent in
the dialog box to provide a name for a user.

Updated based off information on SwitchBrew.

Cleans up unused includes and trims off some dependencies on externals.

* get rid of boost::optional

* Remove optional references

* Use std::reference_wrapper for optional references

* Fix clang format

* Fix clang format part 2

* Adressed feedback

* Fix clang format and MacOS build

Gets rid of the need to call the getter and then check for null.

Nothing requires the shared owner ship here, so we can just return a
plain pointer.

Many of the Current<Thing> getters (as well as a few others) were
missing const qualified variants, which makes it a pain to retrieve
certain things from const qualified references to System.

* remove unnecessary if-statements

* Addressed feedback

Crypto revisions are hex numbers and this function only checks if the string is valid for stoul in base 16, so it should be isxdigit.

Prevents nullptr bug when trying to dump the RomFS of an NSP resulting from secondary_loader not being initialized.

Nothing from this enum is intended to be used outside of this function.

This retrieves:

if (curr_thread == handle_thread) {
   result = total_thread_ticks + (hardware_tick_count - last_context_switch_ticks);
} else if (curr_thread == handle_thread && sub_id == current_core_index) {
   result = hardware_tick_count - last_context_switch_ticks;
}

- This is an incomplete implementation. It was tested with Super Mario Party.

Kernel/Memory: Added a function to first a suitable guest address at which to allocate a region of a given size.

Prevents a potential bug when using RLE records in an IPS patch.

Updated based off the information provided by Hexkyz on Switchbrew.

This is just flat data, so it doesn't really need to be in the function
itself. This also allows deduplicating the constant for the backup size
in GetImageSize().

Silences compiler warnings related to truncation. This also introduces a
small helper function to perform the clamping of the image size.

Allows unindenting the other branch's code.

Now that we've gotten the innaccurate error codes out of the way, we can
finally toss away a bunch of these, trimming down the error codes to
ones that are actually used and knocking out two TODO comments.

This is more consistent with what the kernel does.

This is what the kernel does in this instance.

These are now entirely unused and can be removed.