Skip to content

Conversation

holub
Copy link
Contributor

@holub holub commented Apr 28, 2022

Makes ula128 test work for spectrum 128, +2 models
+2A, +3 uses same pattern as models above but possibly this test not expected to work on there models - nice to have confirmation from owners of physical devices.

ula128_pls2

compare to master:
ula128_pls2_before

@cuavas
Copy link
Member

cuavas commented Apr 28, 2022

Please be a bit more descriptive in pull request titles and commit messages. It’s far better when dealing with the history if the main thing affected is more immediately apparent. “128+ models” doesn’t make sense without context, and “initial contention” on its own doesn’t really describe what this does.

@cuavas cuavas changed the title initial contention for 128+ models spec128.cpp, specpls3.cpp: Added initial bus contention emulation for Spectrum 128+ models Apr 28, 2022
@rb6502 rb6502 merged commit 6032be4 into mamedev:master Apr 30, 2022
@holub holub deleted the spec128-contended branch May 25, 2022 22:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants