[Bug 793751] New: qtdemux_parse_segments: remove superfluous variable

GStreamer (GNOME Bugzilla) bugzilla at gnome.org
Fri Feb 23 12:50:31 UTC 2018


https://bugzilla.gnome.org/show_bug.cgi?id=793751

            Bug ID: 793751
           Summary: qtdemux_parse_segments: remove superfluous variable
    Classification: Platform
           Product: GStreamer
           Version: unspecified
                OS: All
            Status: NEW
          Severity: normal
          Priority: Normal
         Component: gst-plugins-good
          Assignee: gstreamer-bugs at lists.freedesktop.org
          Reporter: aboya at igalia.com
        QA Contact: gstreamer-bugs at lists.freedesktop.org
     GNOME version: ---

`count` and `i` are used interchangeably, why not use a single variable?

The patch attached does that, renaming both to segment_number.

Additionally, at the end of the loop, usages of segment_number are changed to
n_segments for better readability (segment_number is a loop index and should
not be used outside of the loop in presence of better alternatives).

-- 
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