| Age | Commit message (Collapse) | Author |
|---|
|
This fourth part of my series of patches to get rid of unsafe uses of
GetPointer takes care of the "easy" cases in VideoCommon. Three uses of
GetPointer now remain in Dolphin: VertexLoaderManager, TextureInfo, and
the software renderer's TextureSampler.
|
|
|
|
Makes the use of the interface a little less noisy, especially given
how much of the interface depends on an instance being present.
|
|
detected in places other than VertexShaderManager
|
|
|
|
|
|
|
|
|
|
|
|
DataReader is generally jank - it has a start and end pointer, but the end pointer is generally not used, and all of the vertex loaders mostly bypassed it anyways.
Wrapper code (the vertex loaer test, as well as Fifo.cpp and OpcodeDecoding.cpp) still uses it, as does the software vertex loader (which is not a subclass of VertexLoader). These can probably be eliminated later.
|
|
A bit of a micro optimization:
CheckCPConfiguration is called 350 times instead of 35k times.
|
|
|
|
|
|
That way, they're in the same place the panic alerts are generated.
|
|
A pop-up is no longer generated for the Wiggler capsule in Mario Party 5 (https://bugs.dolphin-emu.org/issues/8104).
|
|
|
|
|
|
|
|
|
|
|
|
|
|
Fixes Bomberman Jetters in single core mode.
When single core mode pauses the CPU to execute the GPU
FIFO it greedily executes the whole thing. Before this commit,
Finish and Token interrupts would happen instantly, not even
taking into account how long the current FIFO window has
taken to execute. The interrupts would be effectively backdated
to the start of this execution window.
This commit does two things: It pipes the current FIFO window
execution time though to the interrupt scheduling and it enforces
a minimum delay of 500 cycles before an interrupt will be fired.
|
|
SPDX standardizes how source code conveys its copyright and licensing
information. See https://spdx.github.io/spdx-spec/1-rationale/ . SPDX
tags are adopted in many large projects, including things like the Linux
kernel.
|
|
Migrates off the printf-based formatting where applicable.
|
|
We can make use of if constexpr in several scenarios here to allow
compilers to exise the relevant code paths out.
Technically a decent compiler would do this already, but now we can give
compilers a little more nudging here in the event that isn't the case.
|
|
Nothing provided by these headers are used, so we can remove them.
|
|
cmd2 is a u32, so any bitwise arithmetic on it with a type of the same
size or smaller will result in a u32 value. This is also implicitly
converted to an unsigned type in the if statement as well, given that
size_t * int -> size_t.
This is just more explicit about the operations occurring and also
likely silences a sign conversion warning.
|
|
With the use of a lambda and a change in switch fallthrough, we can
completely eliminate the use of goto within Run().
|
|
Provides a region for all internal utilities.
|
|
Amends a documentation comment that acquired some wonky formatting
during the introduction of clang-format a few years ago.
|
|
Keeps the global localized with the code that it's primarily related to.
Now it's obvious from a glance what the global variable is affecting.
|
|
Provides consistent naming of variables within the translation unit.
While we're at it, we can mark them const where applicable.
|
|
Makes the global variable follow our convention of prefixing g_ on
global variables to make it obvious in surrounding code that it's not a
local variable.
|
|
Rather than making Statistics' member functions operate on the global
variable instance of itself, we can make these functions member
functions and operate on a by-instance state, removing the direct
dependency on the global variable itself.
This also makes for less reading, as there's no need to repeat "stats."
for all variable accesses.
|
|
Normalizes all variables related to statistics so that they follow our
coding style.
These are relatively low traffic areas, so this modification isn't too
noisy.
|
|
It's only used as an interface between two classes. So no need to declare
it in the backend export header.
|
|
|
|
Most of this commits changes performance decreasing logs from info to debug and also cleans up innacurate levels.
|
|
|
|
|
|
|
|
This is only ever queried, making it a global isn't necessary.
|
|
|
|
Also remedies places where the video backends and core rely on things
being indirectly included.
|
|
|
|
|
|
|
|
|
|
|
|
|
