[Bug 727878] qtdemux: remove duplicated code when parsing trex atom

GStreamer (bugzilla.gnome.org) bugzilla at gnome.org
Wed Apr 9 01:01:53 PDT 2014


https://bugzilla.gnome.org/show_bug.cgi?id=727878
  GStreamer | gst-plugins-good | 1.x

Sebastian Dröge (slomo) <slomo> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #273854|none                        |needs-work
             status|                            |

--- Comment #2 from Sebastian Dröge (slomo) <slomo at coaxion.net> 2014-04-09 08:01:45 UTC ---
(From update of attachment 273854)
See the comment above the first one. It reads the sample description index,
then the second one reads the duration into the same variable and overwrites
it.

Maybe change the first one to write to a dummy variable that is not used
elsewhere... for clarity. But the code is correct

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.


More information about the gstreamer-bugs mailing list