| Age | Commit message (Collapse) | Author |
|---|
|
|
|
Recent changes to the build system that made more warnings be flagged as
errors caused building via clang to break.
Fixes #4795
|
|
Makes our error coverage a little more consistent across the board by
applying it to Linux side of things as well. This also makes it more
consistent with the warning settings in other libraries in the project.
This also updates httplib to 0.7.9, as there are several warning
cleanups made that allow us to enable several warnings as errors.
|
|
file_sys/nsp: Make SetTicketKeys actually do something
|
|
|
|
Allows some implementations to avoid completely zeroing out the internal
buffer of the optional, and instead only set the validity byte within
the structure.
This also makes it consistent how we return empty optionals.
|
|
Prevents unnecessary copying of the line being parsed.
|
|
submission_package: Account for multi-content NSPs
|
|
Previously we assumed a submission package can only contain one Program NCA with a single TitleID.
However, Super Mario 3D All-Stars contains four Program NCAs, each with their unique TitleIDs.
This accounts for the existence of multi-content games such as this one.
- Fixes booting Super Mario 3D All-Stars from the games list.
|
|
This isn't used anywhere, so this is a trivial fix.
|
|
|
|
|
|
nca_patch: Reduce stack usage size within SearchBucketEntry()
|
|
Previously this function was using ~16KB of stack (16528 bytes), which
was caused by the function arguments being taken by value rather than by
reference.
We can make this significantly lighter on the stack by taking them by
reference.
|
|
This is only used internally and doesn't depend on any class state, so
we can make it fully internal.
|
|
This isn't used within the function at all in any implementations, so we
can remove it entirely.
|
|
We make it explicit that we're truncating arithmetic here to resolve
compiler warnings (even if the sizes weren't u32/u64 arithmetic
generally promotes to int :<)
|
|
We can use these to avoid typing the same type redundantly. This way, if
these ever change, only a single location needs to be modified.
|
|
These functions are only used within this translation unit, so we can
make them internally linked.
|
|
Fixes build issues
|
|
file_sys: Replace inclusions with forward declarations where applicable
|
|
registered_cache: Make use of designated initializers
|
|
Same behavior, minus unnecessary inclusions where not necessary.
|
|
Simplifies code.
|
|
Removes the need for comments to indicate the fields being assigned.
|
|
|
|
Avoids some trivially avoidable map lookups by keeping the result of
find operations around and querying them.
|
|
Migrates a remaining common file over to the Common namespace, making it
consistent with the rest of common files.
This also allows for high-traffic FS related code to alias the
filesystem function namespace as
namespace FS = Common::FS;
for more concise typing.
|
|
|
|
vfs_vector: Make creation of array vfs files less verbose
|
|
aes_util: Allow SetIV() to be non-allocating
|
|
We can add a helper function to make creation of these files nicer.
While we're at it, we can eliminate an unnecessary std::array copy in
the constructor. This makes the overhead on some of these functions way
less intensive, given some arrays were quite large.
e.g. The timezone location names are 9633 bytes in size.
|
|
time_zone_binary: Make use of designated initializers
|
|
patch_manager: Resolve -Wignored-qualifier warnings
|
|
fsp-srv: Stub Read/WriteSaveDataFileSystemExtraDataWithMaskBySaveDataAttribute
|
|
In a few places, the data to be set as the IV is already within an array.
We shouldn't require this data to be heap-allocated if it doesn't need
to be. This allows certain callers to reduce heap churn.
|
|
registered_cache: Resolve -Wmaybe_uninitialized warnings
|
|
file_sys/mode: Make use of DECLARE_ENUM_FLAG_OPERATORS with Mode
|
|
|
|
Same behavior, minus a hand-rolled operator.
|
|
Top level const will always be ignored in this case, so it can be
removed.
|
|
While we're at it, we can avoid a redundant map lookup.
|
|
|
|
Fixes an access violation where the file no longer exists at the specified path while being parsed.
|
|
|
|
Previously, the method wasn't modifying any class state and therefore not having any effects when called.
Since this has been the case for a very long time now, I'm not sure if we couldn't just remove this method altogether.
|
|
fs: Fix RomFS building when zero byte files are present
|
|
|
|
|
|
|
