smokephil/ardour
1
0
Fork 0
mirror of https://github.com/Ardour/ardour.git synced 2026-01-30 00:43:18 +01:00
Code Issues Projects Releases Wiki Activity

libsmf: do not add a non-zero EndOfTrack event at end of all-zero tracks

Browse source 
This can lead to confusion when trying to decide if the file contains
an explicit duration, since we end up with a track who last event
has a non-zero delta time from the previous one.
This commit is contained in:
Paul Davis 2026-01-27 13:49:40 -07:00
parent 708d5588f3
commit 88e8dcd583
1 changed files with 21 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

21
libs/evoral/libsmf/smf.c
View file

@ -560,6 +560,27 @@ int
smf_track_add_eot_pulses(smf_track_t *track, size_t pulses)
{
smf_event_t *event, *last_event;
int i;
int all_zero = 1;
/* Do not add EOT events to tracks that have only events at zero */
for (i = 0; i < track->events_array->len; ++i) {
smf_event_t* ev = (smf_event_t*)g_ptr_array_index(track->events_array, i);
if (event->time_pulses != 0) {
all_zero = 0;
break;
}
}
if (all_zero && track->number_of_events != 0) {
/* Do not set the End Of Track event time to what was requested
if this track is full of events at zero. If we do that, we
can end up making it appear that the file has an explicit
duration when it does not.
*/
pulses = 0;
}
last_event = smf_track_get_last_event(track);
if (last_event != NULL) {