After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 586019 - [mxfdemux] stray semicolons after if statement
[mxfdemux] stray semicolons after if statement
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-bad
git master
Other Linux
: Normal trivial
: 0.10.14
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2009-06-16 17:48 UTC by Tim-Philipp Müller
Modified: 2009-06-18 20:22 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
fix stray semicolons (1.08 KB, patch)
2009-06-16 17:48 UTC, Tim-Philipp Müller
committed Details | Review

Description Tim-Philipp Müller 2009-06-16 17:48:29 UTC
Surely these are wrong? No idea what exactly they break though, I'm sure slomo will know ;)
Comment 1 Tim-Philipp Müller 2009-06-16 17:48:55 UTC
Created attachment 136752 [details] [review]
fix stray semicolons
Comment 2 Sebastian Dröge (slomo) 2009-06-18 07:31:26 UTC
The patch is correct, it's just an optimization that isn't used when the semicolon is there ;) At that point the prev_partition fields of the partitions are set to the previous partition if known already (otherwise we have to search later but most MXF files already have this set in the files).

Both parts of the diff are actually doing the same btw ;)