We need to perform explicit casts here, otherwise we're implicitly
truncating a 64-bit type to a 32-bit one.

Without this, the std::move within the constructor initializer list
won't be able to actually perform a move.

Allows us to remove a constructor initializer list.

Given the string is appended to another, we can make it a view so a
redundant full copy of the string isn't made.

This member function doesn't modify internal member state, so it can be
marked const.

On parse errors, we can log out the explanatory string indicating what
the parsing error was, rather than just ignoring the variable and
returning an overly broad error code.

Silence miscellaneous warnings

ci: Add custom titlebars for mainline and patreon

video_core/shader: Minor changes

These can inhibit the ability of a compiler to perform RVO.

Allows the function to be non-allocating in terms of the output string.

Same behavior, but without the need to heap allocate

This is behaviorally equivalent and also fixes a bug where some members
weren't being moved over.

Keeps them alphabetically sorted for readability.

Provides logical symmetry to the interface.

Avoids unnecessary atomic reference count increments and decrements.

Provides const equivalents of data accessors for use within const
contexts.

Shader_IR: Implement a full control flow decompiler for the shader IR.

TextureCache: Add the ability to deduce if two textures are depth on blit.