These are only used within this class, so we can make them private to
keep their use contained. This also gets rid of the pre-Qt5 'slot'
identifier, since Qt 5's connection syntax doesn't require a function to
be declared a slot anymore.

Delete git file

service/usb: Update service function tables

service/acc: Silence compiler truncation warnings

kernel/error: Amend error return code values

rasterizer: Fix Layered Textures Loading and Cubemaps

game_list_worker: Use QString's formatting instead of fmt in FormatPatchNameVersions()

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.

profile_manager: Use std::optional instead of boost::optional

 yuzu/configuration/config: Use a std::unique_ptr for qt_config instead of a raw pointer

time_stretch: Remove unused m_channel_count member variable

Implement PointSize Output Attribute.

graphic_breakpoints: Correct translation of strings in BreakpointModel's data() function

npad: Remove unused controller variable from OnInit()

game_list: Use QFileInfo instead of common's file functions

game_list: Make game list column headers translatable

bootmanager: Use QStringLiteral instead of std::string to represent the window title.

perf_stats: Remove unused variable within DoFrameLimiting()

aoc_u: Make use of previously-unused CheckAOCTitleIDMatchesBase() function

maxwell_3d/decoders: Remove unused variables

file_sys: Remove unused variables

logging/backend: Add missing services to the log filters

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.

Like with the previous change, the kernel doesn't return NOT_AUTHORIZED
here. It returns INVALID_THREAD_PRIORITY.

kernel/svc: Move and correct returned error code for invalid thread priorities in SetThreadPriority()

All priority checks are supposed to occur before checking the validity
of the thread handle, we're also not supposed to return
ERR_NOT_AUTHORIZED here.

The kernel appears to return 0xE601 for this situation. Particularly in
svcWaitSynchronization, svcReplyAndReceive, and svcGetThreadContext

The kernel appears to return 0xF601 for this case.

Using fmt here requires unnecessary string conversions back into
QString. Instead, we can just use QString's formatting and get the end
result of the formatting operation in the proper type.

Now that we can actually use std::optional on macOS, we don't need to
continue using boost::optional here.

tr() will not function properly on static/global data like this, as the
object is only ever constructed once, so the strings won't translate if
the language is changed without restarting the program, which is
undesirable. Instead we can just turn the map into a plain old function
that maps the values to their equivalent strings. This is also lessens
the memory allocated, since it's only allocating memory for the strings
themselves, and not an encompassing map as well.