| Age | Commit message (Collapse) | Author |
|---|
|
Avoids the use of the global accessor in favor of explicitly making the
system a dependency within the interface.
|
|
Gets rid of the largest set of mutable global state within the core.
This also paves a way for eliminating usages of GetInstance() on the
System class as a follow-up.
Note that no behavioral changes have been made, and this simply extracts
the functionality into a class. This also has the benefit of making
dependencies on the core timing functionality explicit within the
relevant interfaces.
|
|
core_timing: Make EmptyTimedCallback a local variable
|
|
The necessity of this parameter is dubious at best, and in 2019 probably
offers completely negligible savings as opposed to just leaving this
enabled. This removes it and simplifies the overall interface.
|
|
Given this is only used in one place, it can be moved closest to its
usage site.
|
|
Places all of the timing-related functionality under the existing Core
namespace to keep things consistent, rather than having the timing
utilities sitting in its own completely separate namespace.
|
|
This is a leftover from the early yuzu days.
We shouldn't log every time when we are drawing by default, so let's change the log level to Trace.
|
|
Port citra-emu/citra#4603: "gdbstub: only let Execute breakpoints write/restore BKPT opcodes into target memory"
|
|
|
|
without title ids
|
|
|
|
|
|
memory
|
|
Converts many of the Find* functions to return a std::optional<T> as
opposed to returning the raw return values directly. This allows
removing a few assertions and handles error cases like the service
itself does.
|
|
This member function doesn't actually modify instance state, so it can
be marked as a const member function.
|
|
This was missed within #2075. Renames the member function to make it
consistent with the rest of the Find* functions.
|
|
hwopus: Implement DecodeInterleaved (the newest variant)
|
|
A holdover from citra, the Horizon kernel on the switch has no
prominent kernel object that functions as a timer. At least not
to the degree of sophistication that this class provided.
As such, this can be removed entirely. This class also wasn't used at
all in any meaningful way within the core, so this was just code sitting
around doing nothing. This also allows removing a few things from the
main KernelCore class that allows it to use slightly less resources
overall (though very minor and not anything really noticeable).
|
|
service: Update function tables
|
|
kernel/wait_object: Devirtualize functions related to manipulating the thread list directly
|
|
thread list directly
No inheritors of the WaitObject class actually make use of their own
implementations of these functions, so they can be made non-virtual.
It's also kind of sketchy to allow overriding how the threads get added
to the list anyways, given the kernel itself on the actual hardware
doesn't seem to customize based off this.
|
|
This implementation just calls the base class variant of the function,
so this isn't necessary.
|
|
This just calls the base variant of the function, so it can be removed.
|
|
This functions almost identically to DecodeInterleavedWithPerfOld,
however this function also has the ability to reset the decoder context.
This is documented as a potentially desirable thing in the libopus
manual in some circumstances as it says for the OPUS_RESET_STATE ctl:
"This should be called when switching streams in order to prevent the
back to back decoding from giving different result from one at a time
decoding."
|
|
This member function doesn't actually modify instance state, so it can
be const-qualified.
|
|
This more accurately describes what the function is actually attempting
to do (it's not a simple trivial getter).
|
|
DecodeInterleavedWithPerfOld
Keeps the logic in one spot for use by both functions.
|
|
This doesn't really offer anything over the use of a direct pointer, so
we can just use that instead.
|
|
Makes non-mutable state more explicit.
|
|
Filled in via information provided by SwitchBrew.
|
|
Updates function tables based off information provided by SwitchBrew
|
|
Updates function tables based off information provided by SwitchBrew
|
|
Updates function tables based off information provided by SwitchBrew.
|
|
Updates function tables based off information provided by SwitchBrew.
|
|
Updates function tables based off information provided by SwitchBrew.
|
|
Updates function tables based off information provided by SwitchBrew
|
|
Updates function tables based off information provided by SwitchBrew.
|
|
Updates the function tables based off information on SwitchBrew. Gets
rid of a swath of unknown names.
|
|
In addition to the default, external, EDID, and internal displays,
there's also a null display provided as well, which as the name
suggests, does nothing but discard all commands given to it. This is
provided for completeness.
|
|
warning
Opening a display isn't really a thing to warn about. It's an expected
thing, so this can be a debug log. This also alters the string to
indicate the display name better.
Opening "Default" display reads a little nicer compared to Opening
display Default.
|
|
|
|
Makes non-mutable state more explicit.
|
|
The built-in set of displays is fixed, so we can utilize an array
instead of a vector here.
|
|
|
|
This is kind of a large hole in the API, given we allow popping signed
values. This fixes that.
|
|
This quite literally functions as a basic setter. No other error
checking or anything (since there's nothing to really check against).
With this, it completes the pm:bm interface in terms of functionality.
|
|
Just minor tidying of interfaces.
|
|
This appears to be a vestigial API function that's only kept around for
compatibility's sake, given the function only returns a success error
code and exits.
Since that's the case, we can remove the stubbed notification from the
log, since doing nothing is technically the correct behavior in this
case.
|
|
Looking into the implementation of the C++ standard facilities that seem
to be within all modules, it appears that they use 7 as a break reason
to indicate an uncaught C++ exception.
This was primarily found via the third last function called within
Horizon's equivalent of libcxxabi's demangling_terminate_handler(),
which passes the value 0x80000007 to svcBreak.
|
|
frontend: Refactor ScopeAcquireWindowContext out of renderer_opengl.
|
