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 546570 - gst_caps_append/merge should not require caps2 to be writable
gst_caps_append/merge should not require caps2 to be writable
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gstreamer (core)
git master
Other Linux
: Normal enhancement
: 0.10.36
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2008-08-06 12:17 UTC by Thijs Vermeir
Modified: 2011-10-27 14:49 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
remove restriction from API (2.13 KB, patch)
2008-08-06 12:17 UTC, Thijs Vermeir 		committed Details | Review

Description Thijs Vermeir 2008-08-06 12:17:00 UTC 
Now there is a restriction in gst_caps_append and gst_caps_merge that the caps2 should be writable, having this restriction in the API is a bad idea. Because it breaks the point of refcounting, is not needed in the API and makes it difficult to write bindings for Vala ;-)

Or is there another reason that this is now required in the API?
Comment 1 Thijs Vermeir 2008-08-06 12:17:46 UTC 
Created attachment 115966 [details] [review]
remove restriction from API
Comment 2 David Schleef 2008-08-06 16:42:48 UTC 
It's part of the ABI.  It can be changed in 0.11.
Comment 3 Tim-Philipp Müller 2008-09-09 21:23:48 UTC 
0.11 => marking as enhancement
Comment 4 Tim-Philipp Müller 2011-10-27 14:49:10 UTC 
> It's part of the ABI.  It can be changed in 0.11.

This has recently been changed in 0.10 actually. I don't see how this breaks ABI in any sense.