From: <bug...@bu...> - 2005-07-30 19:31:49
|
Please DO NOT reply to this by email. All additional comments should be m= ade in the comments box of this bug report. http://bugzilla.gnome.org/show_bug.cgi?id=3D312080 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: gst...@li... ReportedBy: t....@ze... QAContact: gst...@li... CC: all...@bu... 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 cou= rse 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. |
From: <bug...@bu...> - 2005-07-30 19:35:02
|
Please DO NOT reply to this by email. All additional comments should be m= ade in the comments box of this bug report. http://bugzilla.gnome.org/show_bug.cgi?id=3D312080 GStreamer | gstreamer (core) | Ver: HEAD CVS ------- Additional Comments From Tim-Philipp M=FCller 2005-07-30 19:34 -= ------ Note also that it is GST_CLOCK_TIME_IS_VALID (clocktime) so that's also a bit inconsistent with the current API Cheers -Tim ------- You are receiving this mail because: ------- You are the assignee for the bug. You are the QA contact for the bug. |
From: <bug...@bu...> - 2005-07-31 16:49:07
|
Please DO NOT reply to this by email. All additional comments should be m= ade in the comments box of this bug report. http://bugzilla.gnome.org/show_bug.cgi?id=3D312080 GStreamer | gstreamer (core) | Ver: HEAD CVS Christian Kirbach changed: What |Removed |Added -------------------------------------------------------------------------= --- Status|UNCONFIRMED |NEW everconfirmed|0 |1 ------- Additional Comments From Christian Kirbach 2005-07-31 16:49 ----= --- any word from the maintainers on this? ------- You are receiving this mail because: ------- You are the assignee for the bug. You are the QA contact for the bug. |
From: <bug...@bu...> - 2005-08-25 23:34:36
|
Please DO NOT reply to this by email. All additional comments should be m= ade in the comments box of this bug report. http://bugzilla.gnome.org/show_bug.cgi?id=3D312080 GStreamer | gstreamer (core) | Ver: HEAD CVS Thomas Vander Stichele changed: What |Removed |Added -------------------------------------------------------------------------= --- AssignedTo|gstreamer- |wi...@fl... |bu...@li... | ------- You are receiving this mail because: ------- You are the assignee for the bug. You are the QA contact for the bug. |