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 700044 - shapewipe sample pipeline won't work
shapewipe sample pipeline won't work
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-good
git master
Other Linux
: Normal normal
: 1.1.1
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2013-05-10 02:10 UTC by Nicolas Dufresne (ndufresne)
Modified: 2013-05-10 07:27 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
[PATCH] shapewipe: Fix sample pipeline in documentation (1.49 KB, patch)
2013-05-10 02:20 UTC, Nicolas Dufresne (ndufresne)
committed Details | Review
[PATCH] shapewipe: Ensure caps are writable (995 bytes, patch)
2013-05-10 02:21 UTC, Nicolas Dufresne (ndufresne)
committed Details | Review
[PATCH] shapewipe: Can't map twice the same buffer for writing (2.86 KB, patch)
2013-05-10 02:21 UTC, Nicolas Dufresne (ndufresne)
committed Details | Review

Description Nicolas Dufresne (ndufresne) 2013-05-10 02:10:14 UTC
The pipeline itself need fixing as it was not fully ported to 1.0, aslo we should remove undeed (). After fixing that we get critical:

 GStreamer-CRITICAL **: gst_structure_set_name: assertion `IS_MUTABLE (structure)' failed

Patches will follow ...
Comment 1 Nicolas Dufresne (ndufresne) 2013-05-10 02:20:57 UTC
Created attachment 243755 [details] [review]
[PATCH] shapewipe: Fix sample pipeline in documentation


https://bugzilla.gnome.org/show_bug.cgi?id=700044
---
 gst/shapewipe/gstshapewipe.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
Comment 2 Nicolas Dufresne (ndufresne) 2013-05-10 02:21:00 UTC
Created attachment 243756 [details] [review]
[PATCH] shapewipe: Ensure caps are writable


The exist one case where that we endup with original caps in ret, in which
case we are not guaratied to have writable caps. Simply ensure this is the
caps are writable before entering the loop.

https://bugzilla.gnome.org/show_bug.cgi?id=700044
---
 gst/shapewipe/gstshapewipe.c | 1 +
 1 file changed, 1 insertion(+)
Comment 3 Nicolas Dufresne (ndufresne) 2013-05-10 02:21:04 UTC
Created attachment 243757 [details] [review]
[PATCH] shapewipe: Can't map twice the same buffer for writing


I took the opportunity to simplify that code a bit. We now use
gst_buffer_make_writable() to make the buffer writable and map twice the
same buffer, with first map being read/write, and second read only. This
get rid of the critical:

GStreamer-CRITICAL **: gst_structure_set_name: assertion `IS_MUTABLE

https://bugzilla.gnome.org/show_bug.cgi?id=700044
---
 gst/shapewipe/gstshapewipe.c | 26 +++++++-------------------
 1 file changed, 7 insertions(+), 19 deletions(-)
Comment 4 Sebastian Dröge (slomo) 2013-05-10 07:27:21 UTC
All pushed, thanks :)