This should fix crashes when launching multiple games in yuzu

- Fixes an issue where macros could be skipped.
- Fixes rendering of distant objects in Super Mario Odyssey.

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.

Avoids the need to put the scaling parameters all over the place for the
common case. The only other time scaling is done is to generate the
smaller 48x48 image, so this is fine.

Avoids the need to duplicate this all over the place, and makes it
translator-friendly across the board.

We should display an error to the user if setting a user image for an
account fails, rather than continuing onwards.