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 557085 - [videobox] support dynamically changing properties
[videobox] support dynamically changing properties
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-good
0.10.10
Other Linux
: Normal normal
: 0.10.11
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2008-10-20 13:49 UTC by Wim Taymans
Modified: 2008-10-21 12:39 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
patch to fix the problem (3.81 KB, patch)
2008-10-20 13:55 UTC, Wim Taymans
none Details | Review
patch to fix the problem (3.81 KB, patch)
2008-10-20 13:58 UTC, Wim Taymans
committed Details | Review

Description Wim Taymans 2008-10-20 13:49:54 UTC
Videobox does not handle dynamically changing the boxing properties because it messes with gst_base_transform_set_passthrough() in the wrong place.
Comment 1 Wim Taymans 2008-10-20 13:55:57 UTC
Created attachment 120928 [details] [review]
patch to fix the problem

Patch to update:

 - debug info
 - don't call gst_base_transform_set_passthrough() in the set_property method
 - inline obsolete method back in setcaps.
Comment 2 Wim Taymans 2008-10-20 13:58:45 UTC
Created attachment 120929 [details] [review]
patch to fix the problem

Patch to update:

 - debug info
 - don't call gst_base_transform_set_passthrough() in the set_property method
 - inline obsolete method back in setcaps.
Comment 3 Jan Schmidt 2008-10-21 12:39:22 UTC
Changed my mind about putting this in. Committed:

         * gst/videobox/gstvideobox.c:
        support dynamically changing properties in videobox
        Fixed: #557085
        Patch By: Wim Taymans <wim.taymans@collabora.co.uk>