[gstreamer-bugs] [Bug 312080] New: [0.9] [API] GST_BUFFER_FOO_IS_VALID(buf) is bad API

[bugzilla-daemon at bugzilla.gnome.org]

Please DO NOT reply to this by email. All additional comments should be made in
the comments box of this bug report.

 http://bugzilla.gnome.org/show_bug.cgi?id=312080
 GStreamer | gstreamer (core) | Ver: HEAD CVS

           Summary: [0.9] [API] GST_BUFFER_FOO_IS_VALID(buf) is bad API
           Product: GStreamer
           Version: HEAD CVS
          Platform: Other
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: Normal
         Component: gstreamer (core)
        AssignedTo: gstreamer-bugs at lists.sourceforge.net
        ReportedBy: t.i.m at zen.co.uk
         QAContact: gstreamer-bugs at lists.sourceforge.net
                CC: all-bugs at bugzilla.gnome.org


Currently we have a couple of macros like this:

  GST_BUFFER_OFFSET_IS_VALID (buffer)


That is bad API IMHO and should rather be:

  GST_BUFFER_OFFSET_IS_VALID (offset)
    and
  GST_BUFFER_HAS_VALID_OFFSET (buffer)


(I'd be willing to fix gstreamer, gst-plugins-base and gst-plugins of course if
people agree that the suggested API is better).

 Cheers
  -Tim

------- You are receiving this mail because: -------
You are the assignee for the bug.
You are the QA contact for the bug.