| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0001504 | OpenMPT | File Format Support | public | 2021-09-11 14:05 | 2021-09-12 22:47 |
| Reporter | StarWolf3000 | Assigned To | Saga Musix | ||
| Priority | normal | Severity | minor | Reproducibility | always |
| Status | resolved | Resolution | fixed | ||
| Platform | 64-bit | OS | Windows | OS Version | 8.1 |
| Product Version | OpenMPT 1.29.12.00 / libopenmpt 0.5.11 (upgrade first) | ||||
| Target Version | OpenMPT 1.29.13.00 / libopenmpt 0.5.12 (upgrade first) | Fixed in Version | OpenMPT 1.29.13.00 / libopenmpt 0.5.12 (upgrade first) | ||
| Summary | 0001504: First DLS instrument region is not imported | ||||
| Description | OpenMPT 1.29.12.00 seems to have introduced some broken behavior on how some imported instruments in the higher patch numbers from SF2/DLS are handled. Just playing back the same instruments in both 1.29.11.00 and 1.29.12.00 doesn't play the same samples, even regions, of the instruments, and even loads some "garbage" in the newer version (see attached pictures). Samples load correctly in 1.29.11.00. | ||||
| Steps To Reproduce | 1) Use 1.29.12.00 2) Create empty module or load one 3) "Import" one of the shown instruments from GM.DLS (see one of the pictures which one are affected) 4) Look in the Sample tab and/or play back the selected instrument/sample | ||||
| Tags | No tags attached. | ||||
| Has the bug occurred in previous versions? | No | ||||
| Tested code revision (in case you know it) | |||||
 |
Might be related to the changes introduced in 0001457 |
 |
Instead of just taking the first matching region, that change made OpenMPT instead consider the smallest (most specific) region covering the requested key range. In those instruments, there is a bogus 1-key region spanning just the lowest note, so that is chosen. I guess I'll have to finetune that algorithm a bit more. |
|
|
Upon closer inspection, it seems to be unrelated to that, and it looks more like something in the importing of instrument regions broke when adding support for more than 128 regions per instrument. Every DLS instrument is missing its first region and has a bogus last region. Those instruments you mentioned only have one region, so you only end up with that bogus region. |
|
|
Should be fixed in r15678. |
