Opened 10 years ago

Closed 10 years ago

Last modified 19 months ago

#9188 closed patch

rjp1: calculate sample length correctly

Reported by: fuzzie Owned by: fingolfin
Priority: normal Component: Audio
Keywords: Cc:
Game:

Description

This fixes the RJP1 player to use the correct sample length - the length entry in the file is actually "length after provided position", it seems.

I couldn't find any specs, but I looked at the m68k player source I could find for the format, made this patch based on what it seemed it was doing, and then thought to check the start/end offsets within the data: they match the reasoning of this patch (i.e. the sample data is packed together if at this length) and the data seems correct, so I'm pretty sure this is correct.

This fixes bug 3001110, which was due to an assert since the playback was starting after the supposed end of samples. Patch against trunk.

Ticket imported from: #3006178. Ticket imported from: patches/1293.

Attachments (1)

0001-rjp1-calculate-sample-length-correctly.patch (781 bytes ) - added by fuzzie 10 years ago.

Download all attachments as: .zip

Change History (4)

comment:1 by fingolfin, 10 years ago

Owner: set to fingolfin
Status: newclosed

comment:2 by fingolfin, 10 years ago

Thanks, committed in r49210.

comment:3 by digitall, 19 months ago

Component: Audio
Note: See TracTickets for help on using tickets.