kernel/server_session: Remove obsolete TODOs

These are holdovers from Citra.

Remove unnecessary bounding in LD_C

yuzu/debugger: Remove graphics surface viewer

video_core/texures/texture: Remove unnecessary includes

file_sys: Provide generic interface for accessing game data

Correct XMAD mode, psl and high_b on different encodings.

Port citra-emu/citra#4437: "citra-qt: Make hotkeys configurable via the GUI (Attempt 2)"

yuzu/loading_screen: Resolve runtime Qt string formatting warnings

gl_backend: Align Pixel Storage

Correct LOP_IMM encoding

kernel/process: Set page table when page table resizes occur.

We need to ensure dynarmic gets a valid pointer if the page table is
resized (the relevant pointers would be invalidated in this scenario).

In this scenario, the page table can be resized depending on what kind
of address space is specified within the NPDM metadata (if it's
present).

In our error console, when loading a game, the strings:

QString::arg: Argument missing: "Loading...", 0
QString::arg: Argument missing: "Launching...", 0

would occasionally pop up when the loading screen was running. This was
due to the strings being assumed to have formatting indicators in them,
however only two out of the four strings actually have them.

This only applies the arguments to the strings that have formatting
specifiers provided, which avoids these warnings from occurring.

This commit makes sure GL reads on the correct pack size for the
respective texture buffer.

shader_cache: Permit a Null Shader in case of a bad host_ptr.

maxwell_3d: Reduce severity of ProcessSyncPoint

Port citra-emu/citra#4718: "fix clang-format target when using a path with spaces on windows"

shader_ir: Implement AOFFI for TEX and TLD4

core/memory: Minor simplifications to page table management

gl_state: Rework to enable individual applies

gl_shader_disk_cache: Use Zstandard for compression

kernel/{server_port, server_session}: Return pairs instead of tuples from pair creation functions

core/memory: Remove unused enum constants

memory_manager: Improved implementation of read/write/copy block.

These are holdovers from Citra and can be removed.

Now that nothing actually touches the internal page table aside from the
memory subsystem itself, we can remove the accessor to it.

Given the page table will always be guaranteed to be that of whatever
the current process is, we no longer need to keep this around.

Centralizes the page table switching to one spot, rather than making
calling code deal with it everywhere.

Keeps the return type consistent with the function name. While we're at
it, we can also reduce the amount of boilerplate involved with handling
these by using structured bindings.

Returns the same type that the function name describes.

Avoids dragging in a direct dependency in a header.

Nothing in this header relies on common_funcs or the memory manager.

This gets rid of reliance on indirect inclusions in the OpenGL caches.

This doesn't actually work anymore, and given how long it's been left in
that state, it's unlikely anyone actually seriously used it.

Generally it's preferable to use RenderDoc or Nsight to view surfaces.