Skip to content

Fix XG Drum bank selection#1514

Merged
derselbst merged 2 commits intomasterfrom
1508
Mar 29, 2025
Merged

Fix XG Drum bank selection#1514
derselbst merged 2 commits intomasterfrom
1508

Conversation

@derselbst
Copy link
Copy Markdown
Member

A regression introduced in #1486 causes the bankmsb setter to not update the bank for drum channels. This fixes this behavior, by applying the bankMsb as bank number in case it's an XG drum bank.

Fixes #1508

@derselbst derselbst added the bug label Mar 22, 2025
@derselbst derselbst added this to the 2.4 milestone Mar 22, 2025
@sonarqubecloud
Copy link
Copy Markdown

@derselbst derselbst merged commit dc939b8 into master Mar 29, 2025
75 of 79 checks passed
@derselbst derselbst deleted the 1508 branch March 29, 2025 13:30
DominusExult added a commit to DominusExult/fluidsynth-sans-glib that referenced this pull request Apr 7, 2025
* master: (51 commits)
  Put fluidsynth.lock in its own directory (FluidSynth#1528)
  Fix XG Drum bank selection (FluidSynth#1514)
  Stop SDL3 intercepting signals (FluidSynth#1520)
  Fix MSVC type truncation warnings (FluidSynth#1515)
  Fix SDL3.dll not included in prebuilt Windows Binaries (FluidSynth#1518)
  Fix Altitude test case
  Update testcases for FluidSynth#1508
  Apply *_FOR_BUILD environment variables to gentables (FluidSynth#1501)
  Add testfiles for FluidSynth#1496 and FluidSynth#1505
  Remove unused enum
  Bump to 2.4.4
  Fix ordering and dependencies in fluidsynth.service file (FluidSynth#1500)
  fluidsynth.1: Some remarks and a patch with editorial changes for this man page (FluidSynth#1499)
  Add support for SDL3 (FluidSynth#1485)
  Fix interruptions of real-time playback (FluidSynth#1494)
  Add two more test cases
  Log SysEX System ON Messages
  Return error code on server failure
  Rephrase error messages for clarity
  Only run one FluidSynth service at a time
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

XG Drums broken in 2.4.4

1 participant