Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

seibu/sei021x_sei0220_spr.cpp: Device-fied SEI0210/SEI0211/SEI0220 sprite emulation #11802

Merged
merged 5 commits into from
Dec 7, 2023

Conversation

cam900
Copy link
Contributor

@cam900 cam900 commented Nov 30, 2023

seibu/banprestoms.cpp: Use single-pass sprite drawing

seibu/bloodbro.cpp: Separate class via hardware configurations, Reduce duplications

seibu/dcon.cpp: Fix sprite coordinate format, fix GFX ROM region naming

seibu/goodejan.cpp: Fix comment

seibu/sengokmj.cpp: Use single-pass sprite drawing

…I0220 sprite emulation

banprestoms.cpp: Use single-pass sprite drawing

bloodbro.cpp: Sepatate class via hardware configurations

dcon.cpp: Fix sprite coordinate format

goodejan.cpp: Fix comment

sengokmj.cpp: Use single-pass sprite drawing
@happppp
Copy link
Member

happppp commented Nov 30, 2023

I haven't looked at the code, but let me comment that you chose a very long filename.
If for example SEI0212 and SEI0213 existed in this chip family, it'd be named "sei0210_sei0211_sei0212_sei0213_sei0220_spr.cpp"?

Normally with a device family, you name the file after the base device.

@startaq
Copy link
Member

startaq commented Nov 30, 2023

Agree with hap. Sometimes we also use a placeholder, so it could also potentially be sei02xx_spr.cpp.

@cam900 cam900 changed the title seibu/sei0210_sei0211_sei0220_spr.cpp: Device-fied SEI0210/SEI0211/SEI0220 sprite emulation seibu/sei021x_sei0220_spr.cpp: Device-fied SEI0210/SEI0211/SEI0220 sprite emulation Dec 1, 2023
src/mame/seibu/sei021x_sei0220_spr.h Outdated Show resolved Hide resolved
src/mame/seibu/sei021x_sei0220_spr.h Outdated Show resolved Hide resolved
src/mame/seibu/sei021x_sei0220_spr.cpp Outdated Show resolved Hide resolved
@cuavas cuavas merged commit 0a0ddfb into mamedev:master Dec 7, 2023
5 checks passed
Luigi30 pushed a commit to Luigi30/mame that referenced this pull request Dec 16, 2023
…prite chip emulation. (mamedev#11802)

* seibu/banprestoms.cpp, seibu/sengokumj.cpp: Use single-pass sprite drawing.
* seibu/bloodbro.cpp: Split driver state classes for different hardware configurations.
* seibu/dcon.cpp: Fixed sprite coordinate format.
einstein95 pushed a commit to einstein95/mame that referenced this pull request Mar 2, 2024
…prite chip emulation. (mamedev#11802)

* seibu/banprestoms.cpp, seibu/sengokumj.cpp: Use single-pass sprite drawing.
* seibu/bloodbro.cpp: Split driver state classes for different hardware configurations.
* seibu/dcon.cpp: Fixed sprite coordinate format.
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.

None yet

4 participants