 |
It seems like the NoiseTracker detection might be a bit too aggressive here and even apply to very basic tracks. Noise Tracker treats all Dxx commands as D00, so if OpenMPT thinks it is likely that the module was made with NoiseTracker it will do the same.
|
 |
I see... Trying to have compatibility with everything can be a bit of a pain, I know that firsthand as a developer myself. In that case, I'm unsure if a fix is possible, but thanks for telling me about these workarounds. |
|
|
I will have to do a bit more research, but depending on the amount of NoiseTraker modules that actually need this fix for the Dxx command, it may not be worth the effort to keep the heuristic at all. |
|
|
The heuristic has been finetuned in r20486 so fewer modules should be affected by it. The improvement will only work if there are any empty sample slots like in this particular module, though, but it's the only thing I could come up with for now. Updated test builds will be available in a few hours from https://builds.openmpt.org/builds/ |
|
|
It appears to be working -- the Dxx values are now being loaded correctly. |
- Anonymous
