qtmux: Remove bogus check in atom_stsc_add_new_entry()
authorJan Schmidt <jan@centricular.com>
Fri, 30 Dec 2016 11:31:38 +0000 (22:31 +1100)
committerJan Schmidt <jan@centricular.com>
Mon, 2 Jan 2017 14:34:02 +0000 (01:34 +1100)
Remove an old check from atom_stsc_add_new_entry() that
extends the last entry in the STSC if the samples per chunk
matches, as the new interleave merging logic requires that
the final entry by updateable. There's already code
below which simply merges the final entry into the previous
one when needed, so rely on that instead.

Fixes asserts like:

ERROR:atoms.c:2940:atom_stsc_update_entry: assertion failed:
(atom_array_index (&stsc->entries, len - 1).first_chunk == first_chunk)

gst/isomp4/atoms.c

index 0747f9bbc3ada98dc7e72d80853a841199657a64..d11033349019d74bf53c8ddbb0ed8b641aaa7e49 100644 (file)
@@ -2904,11 +2904,6 @@ atom_stsc_add_new_entry (AtomSTSC * stsc, guint32 first_chunk, guint32 nsamples)
 {
   gint len;
 
-  if ((len = atom_array_get_len (&stsc->entries)) &&
-      ((atom_array_index (&stsc->entries, len - 1)).samples_per_chunk ==
-          nsamples))
-    return;
-
   if ((len = atom_array_get_len (&stsc->entries)) > 1 &&
       ((atom_array_index (&stsc->entries, len - 1)).samples_per_chunk ==
           (atom_array_index (&stsc->entries, len - 2)).samples_per_chunk)) {