Skip to content

Conversation

cam900
Copy link
Contributor

@cam900 cam900 commented Oct 2, 2025

seibu/raiden2.cpp: Minor cleanups:

  • Fix bankswitching in reset
  • Simplify CPU bankswitching method
  • Use logmacro.h for logging
  • Fix save state supports
  • Reduce preprocessor defines
  • Make some variables constant
  • Use generic gfx decode layout
  • Fix CPU ROM region size
  • Use BIT macro for single bit values
  • Reduce runtime tag lookups
  • Fix namings
  • Fix spacings
  • Fix filenames
  • Add notes

seibu/r2dx_v33.cpp: Minor cleanups:

  • Split driver states per configurations
  • Fix save state support
  • Fix bankswitching in reset
  • Use BIT macro for single bit value
  • Use hexadecimal value for address value
  • Reduce runtime tag lookups
  • Add ROMREGION_ERASEFF for regions to prevent possible transparency issue
  • Fix spacings
  • Fix namings

seibu/seibu_helper.cpp: Split partial_carry_sum into seibu_helper.cpp for reduce including unnecessary functions

seibu/r2crypt.cpp: Allow variable ROM region and size in sprite ROM decryption functions

seibu/seibuspi_m.cpp, seibu/seibuspi_v.cpp: Fix spacings

seibu/legionna.cpp: Fix naming

cam900 and others added 6 commits October 2, 2025 20:53
- Fix bankswitching in reset
- Simplify CPU bankswitching method
- Use logmacro.h for logging
- Fix save state supports
- Reduce preprocessor defines
- Make some variables constant
- Use generic gfx decode layout
- Fix CPU ROM region size
- Use BIT macro for single bit values
- Reduce runtime tag lookups
- Fix namings
- Fix spacings
- Fix filenames
- Add notes

seibu/r2dx_v33.cpp: Minor cleanups:
- Split driver states per configurations
- Fix save state support
- Fix bankswitching in reset
- Use BIT macro for single bit value
- Use hexadecimal value for address value
- Reduce runtime tag lookups
- Add ROMREGION_ERASEFF for regions to prevent possible transparency issue
- Fix spacings
- Fix namings

seibu/seibu_helper.cpp: Split partial_carry_sum into seibu_helper.cpp for reduce including unnecessary functions

seibu/r2crypt.cpp: Allow variable ROM region and size in sprite ROM decryption functions

seibu/seibuspi_m.cpp, seibu/seibuspi_v.cpp: Fix spacings

seibu/legionna.cpp: Fix naming
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These functions should really have less generic names to avoid potential conflicts.

@cuavas cuavas merged commit a42e027 into mamedev:master Oct 6, 2025
7 checks passed
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.

2 participants