View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0000117 | OpenMPT | General | public | 2011-04-21 23:17 | 2011-05-31 18:06 |
Reporter | 404notfound | Assigned To | Saga Musix | ||
Priority | normal | Severity | minor | Reproducibility | always |
Status | resolved | Resolution | fixed | ||
Platform | x86 | OS | Windows | OS Version | 7 |
Product Version | OpenMPT 1.19.01.00 (upgrade first) | ||||
Target Version | OpenMPT 1.19.02.00 (upgrade first) | Fixed in Version | OpenMPT 1.19.02.00 (upgrade first) | ||
Summary | 0000117: Incorect length in 8SVX samples | ||||
Description | The attached samples(amiga IFF I believe) and other similar ones get imported as 0 bytes long for some odd reason. Samples taken from the ST-XX. | ||||
Steps To Reproduce | 1.Import sample.
| ||||
Tags | No tags attached. | ||||
Attached Files | |||||
Has the bug occurred in previous versions? | I believe it's an old bug. | ||||
Tested code revision (in case you know it) | |||||
The 8SVX headers for those samples are broken - they contain wrong size information. So OpenMPT is actually behaving correctly. "gatedbassdrum" and "techno1" claim to contain a sample that is larger than the file itself, and in "ohrfeige" the sample length is specified to be 0. |
|
Maybe have something like "Internal data size is incorrect. Open anyway?". |
|
I can make it trim the length if there is not enough data available, but I don't really know if making a guess when the "data length" field is 0 is a good idea at all... (Reason: The specs don't say that 0 is an invalid block length, so this would break files with actual 0-byte blocks - why should we comply to broken files instead of working files?) |
|
Or offer to import as raw file if invalid. |
|
Revision 861 should treat 0 byte IFF blocks correctly. |
|
Please check if http://sagagames.de/stuff/mptrack.exe fixes the problem for you. |
|
Confirmed. |
|
Date Modified | Username | Field | Change |
---|---|---|---|
2011-04-21 23:17 | 404notfound | New Issue | |
2011-04-21 23:17 | 404notfound | File Added: st-herpderp.zip | |
2011-04-22 11:03 | 404notfound | Steps to Reproduce Updated | |
2011-04-29 21:41 | Saga Musix | Summary | Incorect length in amiga IFF samples => Incorect length in 8SVX samples |
2011-04-29 21:50 | Saga Musix | Note Added: 0000176 | |
2011-05-02 00:28 | 404notfound | Note Added: 0000177 | |
2011-05-02 10:29 | Saga Musix | Note Added: 0000178 | |
2011-05-02 12:13 | Saga Musix | Note Edited: 0000178 | |
2011-05-02 13:42 | 404notfound | Note Added: 0000179 | |
2011-05-02 16:49 | Saga Musix | Assigned To | => Saga Musix |
2011-05-02 16:49 | Saga Musix | Status | new => assigned |
2011-05-02 17:41 | Saga Musix | Note Added: 0000180 | |
2011-05-03 21:08 | Saga Musix | Note Added: 0000182 | |
2011-05-03 21:08 | Saga Musix | Status | assigned => feedback |
2011-05-03 22:10 | 404notfound | Note Added: 0000184 | |
2011-05-03 22:10 | 404notfound | Status | feedback => assigned |
2011-05-03 22:11 | Saga Musix | Status | assigned => resolved |
2011-05-03 22:11 | Saga Musix | Resolution | open => fixed |
2011-05-03 22:11 | Saga Musix | Fixed in Version | => OpenMPT 1.19.01.* (old testing) |
2011-05-03 22:11 | Saga Musix | Target Version | => OpenMPT 1.19.01.* (old testing) |
2011-05-31 18:06 | Saga Musix | Target Version | OpenMPT 1.19.01.* (old testing) => OpenMPT 1.19.02.00 (upgrade first) |