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

inufuku.cpp: Merge */inufuku.* into drivers/inufuku.cpp, Minor cleanups #8172

Merged
merged 4 commits into from
Jun 12, 2021

Conversation

cam900
Copy link
Contributor

@cam900 cam900 commented Jun 10, 2021

Remove unnecessary VRAM read trampolines
Use buffered_spriteram16_device for buffered sprite RAM
Use clock related to XTALs
Reduce runtime tag lookups, Unnecessary region size
Add logerror for debugging
Fix initialize
Fix namngs, spacings
Use generic gfx layouts, Fix color usage of gfxdecode related to palette amount
Restrict rowscroll with screen cliprect
Reduce unnecessary lines
Use shorter/correct type values

Remove unnecessary VRAM read trampolines
Use buffered_spriteram16_device for buffered sprite RAM
Use clock related to XTALs
Reduce runtime tag lookups, Unnecessary region size
Fix namngs, spacings
Use generic gfx layouts, Fix color usage of gfxdecode related to palette amount
Restrict rowscroll with screen cliprect
Reduce unnecessary lines
Use shorter/correct type values
@startaq
Copy link
Member

startaq commented Jun 10, 2021

Can you move everything to the driver file? It's small enough so that it doesn't need to be split.

Add optional logging for debugging
Fix line usages, Fix initialize
@cam900 cam900 changed the title inufuku.cpp: Minor cleanups inufuku.cpp: Merge */inufuku.* into drivers/inufuku.cpp, Minor cleanups Jun 10, 2021
@cuavas
Copy link
Member

cuavas commented Jun 10, 2021

You may as well put it in an anonymous namespace for the marginal link time improvement while you’re at it.

@cuavas cuavas merged commit 838da5b into mamedev:master Jun 12, 2021
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