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 127133 - [API] Need non-vararg version of gst_index_add_association
[API] Need non-vararg version of gst_index_add_association
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gstreamer (core)
git master
Other All
: High enhancement
: 0.7.5
Assigned To: David Schleef
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2003-11-16 20:19 UTC by Brett Kosinski
Modified: 2005-08-15 01:27 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Proposed solution, defines gst_index_add_association_list (3.92 KB, patch)
2003-11-16 20:20 UTC, Brett Kosinski
none Details | Review

Description Brett Kosinski 2003-11-16 20:19:53 UTC
Basically, we need a version of this function which doesn't take a variable
list of arguments, so that people writing bindings can actually bind this
functionality.
Comment 1 Brett Kosinski 2003-11-16 20:20:44 UTC
Created attachment 21506 [details] [review]
Proposed solution, defines gst_index_add_association_list
Comment 2 Brett Kosinski 2003-11-16 20:22:57 UTC
Just to explain the proposed patch a little.  Basically, it defines
the following function:

GstIndexEntry*
gst_index_add_association_list (GstIndex *index, gint id, 
                                GstAssocFlag flags, 
                                gint nassocs, 
                                GstIndexAssociation *assocs)

Basically, the new version takes an array of GstIndexAssociation,
rather than a variable list of GstFormat/value pairs.

This patch also modifies the original varargs version to call the
_list() equivalent to perform the low-level functionality.
Comment 3 alexander.winston 2004-01-25 07:02:36 UTC
Adding the PATCH keyword and marking priority as high.
Comment 4 Thomas Vander Stichele 2004-02-06 15:50:10 UTC
david, what do you think ?
Comment 5 David Schleef 2004-02-06 18:18:24 UTC
2004-02-06  David Schleef  <ds@schleef.org>

        * gst/gstindex.c: (gst_index_add_format), (gst_index_add_id),
        (gst_index_add_entry), (gst_index_add_associationv),
        (gst_index_add_association): Add gst_index_add_associationv()
        and clean up gst_index_add_association(). #127133