[Bug 657151] ogg: another cleanup round

GStreamer (bugzilla.gnome.org) bugzilla at gnome.org
Tue Aug 23 03:39:43 PDT 2011


https://bugzilla.gnome.org/show_bug.cgi?id=657151
  GStreamer | gst-plugins-base | unspecified

--- Comment #4 from Vincent Penquerc'h <vincent.penquerch at collabora.co.uk> 2011-08-23 10:39:40 UTC ---
Created an attachment (id=194465)
 View: https://bugzilla.gnome.org/attachment.cgi?id=194465
 Review: https://bugzilla.gnome.org/review?bug=657151&attachment=194465

oggdemux: rename local variable for clarity

While the casual reader might end up bewildered by just why this
change might increase clarity, it just happens than, in the libogg
and associated sources, op is the canonical name for an ogg_packet
whlie og is the canonical name for an ogg_page, and reading this
code confuses me.

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