dolphin/Source/Core/VideoCommon/OpcodeDecoding.cpp, branch release-prep-2503a

VideoCommon: Remove calls to GetPointer

2024-04-09T19:08:57+00:00

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.

FifoRecorder: Move instance to System.

2024-01-12T14:06:06+00:00

VideoCommon/CommandProcessor: Pass system instance through constructor

2023-12-20T14:02:53+00:00

Makes the use of the interface a little less noisy, especially given
how much of the interface depends on an instance being present.

VideoCommon: move xf state management to its own class, so changes can be detected in places other than VertexShaderManager

2023-11-26T22:27:26+00:00

VideoCommon/CommandProcessor: Pass System to HandleUnknownOpcode().

2023-04-05T18:09:32+00:00

VideoCommon: De-globalize VertexShaderManager class.

2022-12-28T14:52:29+00:00

VideoCommon/Fifo: Refactor to class, move to Core::System.

2022-12-10T16:16:19+00:00

HW/Memmap: Refactor Memory to class, move to Core::System.

2022-12-03T12:27:02+00:00

VideoCommon/CommandProcessor: Refactor to class, move to Core::System.

2022-11-29T07:15:01+00:00

VertexLoader: Eliminate use of DataReader

2022-11-23T01:17:11+00:00

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.