Given we're already constructing the error code, we don't need to call
the constructor inside of it.

logging: Use std::string_view where applicable

param_package: Minor changes

apm: Improve stub for GetPerformanceConfiguration.

ipc_helpers: Add PushEnum() member function to ResponseBuilder

Allows avoiding string copies by letting the strings be moved into the
function calls.

This avoids a redundant std::string construction if a key doesn't exist
in the map already.

e.g.

data[key] requires constructing a new default instance of the value in
the map (but this is wasteful, since we're already setting something
into the map over top of it).

Avoids potential dynamic allocation occuring during program launch

Allows avoiding constructing std::string instances, since this only
reads an arbitrary sequence of characters.

We can also make ParseFilterRule() internal, since it doesn't depend on
any private instance state of Filter

A few inclusions were being satisfied indirectly. To prevent breakages
in the future, include these directly.

These can just use a view to a string since its only comparing against
two names in both cases for matches. This avoids constructing
std::string instances where they aren't necessary.

Allows pushing strongly-typed enum members without the need to always
cast them at the call sites.

Note that we *only* allow strongly-typed enums in this case. The reason
for this is that strongly typed enums have a guaranteed defined size, so
the size of the data being pushed is always deterministic. With regular
enums this can be a little more error-prone, so we disallow them.

This function simply uses the underlying type of the enum to determine
the size of the data. For example, if an enum is defined as:

enum class SomeEnum : u16 {
  SomeEntry
};

if PushEnum(SomeEnum::SomeEntry); is called, then it will push a
u16-size amount of data.

HLE/ACC: Stub IManagerForApplication::GetAccountId to return an error.

externals: Update glad to version 0.1.25

 gl_state: Get rid of mismatched sign conversions in Apply()

filesys/loader: std::move VirtualFile instances in constructors where applicable

audout_u/audren_u: Ensure null terminators are written out in ListAudioOutsImpl(), ListAudioDeviceName(), and GetActiveAudioDeviceName()

maxwell_3d: Remove unused variable within GetStageTextures()

thread: Convert ThreadStatus into an enum class

partition_filesystem: Return pfs_dirs member variable within GetSubdirectories()

nso: Minor changes

gl_shader_decompiler: Eliminate variable and declaration shadowing

And make IManagerForApplication::CheckAvailability always return false.
Returning a bogus id from GetAccountId causes games to crash on boot.
We should investigate this with a hwtest and either stub it properly or implement it.

Keeps the OpenGL loader library up to date. Previously we were at
version 0.1.16

While we're at it, amend the loop variable type to be the same width as
that returned by the .size() call.

This avoids unnecessary atomic reference count increments and decrements

Avoids potentially unnecessary atomic reference count incrementing and
decrementing, as well as string copying.

gl_shader_decompiler: Remove unnecessary const from return values

pl_u: Simplify WriteBuffer() calls in GetSharedFontInOrderOfPriority()

std::string doesn't include the null-terminator in its data() + size()
range. This ensures that the null-terminator will also be written to the buffer

Uses a type that doesn't potentially dynamically allocate, and ensures
that the name of the interface is properly null-terminated when writing
it to the buffer.

Makes the thread status strongly typed, so implicit conversions can't
happen. It also makes it easier to catch mistakes at compile time.

This should be returned here, otherwise pfs_dirs is effectively only
ever added to, but never read.

Ensures that no identifiers are being hidden, which also reduces
compiler warnings.

This adds nothing from a behavioral point of view, and can inhibit the
move constructor/RVO