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

MidiDriver interface refactor #256

Merged
merged 45 commits into from
Oct 10, 2023
Merged

MidiDriver interface refactor #256

merged 45 commits into from
Oct 10, 2023

Conversation

Raffaello
Copy link
Owner

@Raffaello Raffaello commented Sep 29, 2023

  • IMidiChannel interface
  • implement in the midi drivers the new IMidiChannel interface
  • clean up scummvm midi driver
  • IMidiChannelVoice interface?
  • refactor adlibinstrument and setcustom instrument method (use OPL2Instrument_t)
  • MUS (OplDriver) not sound correctly, opl3 mode looks like no voices on the left, opl2 mode no bass

@Raffaello Raffaello self-assigned this Sep 29, 2023
@Raffaello Raffaello changed the title init MidiDriver interface refactor Sep 29, 2023
@Raffaello Raffaello added the refactor refactoring code, same functionality, but better code label Oct 3, 2023
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 69 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@Raffaello Raffaello merged commit ad5b0f1 into master Oct 10, 2023
6 of 7 checks passed
@Raffaello Raffaello deleted the midi-interface-refactor branch October 10, 2023 12:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactor refactoring code, same functionality, but better code
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

1 participant