summaryrefslogtreecommitdiffstatshomepage
path: root/src/lib/util/jedparse.cpp
diff options
context:
space:
mode:
author Patrick Mackinlay <pmackinlay@hotmail.com>2019-12-18 12:22:15 +0700
committer Patrick Mackinlay <pmackinlay@hotmail.com>2019-12-18 12:22:15 +0700
commit190d1f3ccbfc4b82d5b9a20aa7996bc7073bee29 (patch)
treefe14982d9ef5e19c6ec1ef0cdc210cae5b47cf9e /src/lib/util/jedparse.cpp
parente6133c887159fffd6c52e13a25eb4ffa0635bf37 (diff)
mips1: revert unnecessary/unwanted initialization (nw)
All of these variables are either: 1. Already initialized in the constructor or start/reset; or 2. Guarded against read-before-write by program logic; or 3. Documented as being in an undefined state at device power-up. Regarding the last point, my view is that if the documentation states the value is unpredictable/undefined at power up, I'd rather not initialize it to an arbitrary value. The absence of initialization in the code counts as documenting this behaviour, or conversely, initializing it to an arbitrary zero documents something which is not factual. While there is an argument for consistency in emulation, I'd rather expose and address any such undocumented hardware behaviour. I believe the only required and missing initialization here is for integer register zero, which was indeed a bug. If there are any other test cases which fail after this reversion, please let me know and I'll investigate and address them.
Diffstat (limited to 'src/lib/util/jedparse.cpp')
0 files changed, 0 insertions, 0 deletions