View Issue Details

IDProjectCategoryView StatusLast Update
0001859OpenMPTFile Format Supportpublic2025-01-22 20:18
Reportermusicperson42 Assigned To 
PrioritynormalSeverityminorReproducibilityalways
Status newResolutionopen 
Platformx64OSWindowsOS Version11
Product VersionOpenMPT 1.31.14.00 / libopenmpt 0.7.13 (current stable) 
Summary0001859: Some BRR files do not get imported
Description

Some BRR files do not get imported in the sample editor - the program still detects the samples as raw data to import. I've provided a zip file of those BRR files.

TagsNo tags attached.
Attached Files
BRR samples.zip (340,064 bytes)
Has the bug occurred in previous versions?
Tested code revision (in case you know it)

Activities

Saga Musix

Saga Musix

2025-01-22 20:18

administrator   ~0006303

BRR files have no file header, meaning that there is a high chance that any random binary file is likely to be detected as BRR sample even though it shouldn't be. Hence OpenMPT verifies the integrity of the BRR sample blocks before allowing the file to load. Some of those checks are stricter than what the real hardware requires, and it is known that there are a few samples out there which won't pass this test. In particular, otherwise valid BRR files may get rejected if sample block indicates that the sample should be looped, and then a later sample block claims that the sample shouldn't be looped. This is what causes those files to be rejected.

While real SNES hardware won't care about this inconsistency (it only matters if the last sample block has the loop bit set), this test greatly helps with rejecting files that would otherwise be detected as BRR samples but in reality are just raw data or some other binary sample format. I would be very hesitant to remove this check as these BRR files are just not structurally sound for whatever reason (could be both a bug in the authoring tool or in whatever was used to extract them).

Issue History

Date Modified Username Field Change
2025-01-22 19:43 musicperson42 New Issue
2025-01-22 19:43 musicperson42 File Added: BRR samples.zip
2025-01-22 20:18 Saga Musix Note Added: 0006303