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 325459 - Fractions cannot be used as values for gst.Structure fileds
Fractions cannot be used as values for gst.Structure fileds
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-python
git master
Other Linux
: Normal normal
: 0.10.2
Assigned To: GStreamer Maintainers
Johan (not receiving bugmail) Dahlin
Depends on:
Blocks:
 
 
Reported: 2006-01-01 20:44 UTC by Martin Soto
Modified: 2006-01-08 12:28 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Proposed patch (2.01 KB, patch)
2006-01-01 20:45 UTC, Martin Soto
none Details | Review
Proposed patch, cut against latest gst-python (1.55 KB, patch)
2006-01-02 00:29 UTC, Martin Soto
committed Details | Review

Description Martin Soto 2006-01-01 20:44:11 UTC
Hi,

Trying to set structure fields with gst.Fraction values (common in Gstreamer 0.10) fails with an error. After some investigation, I found that the overriden set funcion for gst.Structure actually doesn't cover this case properly.

The attached patch should fix the problem.
Comment 1 Martin Soto 2006-01-01 20:45:57 UTC
Created attachment 56639 [details] [review]
Proposed patch
Comment 2 Martin Soto 2006-01-02 00:29:05 UTC
Created attachment 56643 [details] [review]
Proposed patch, cut against latest gst-python
Comment 3 Edward Hervey 2006-01-08 12:28:45 UTC
2006-01-08  Martin Soto  <martinsoto@users.sourceforge.net>

	* gst/gststructure.override: (_wrap_gst_structure_set_value):
 	Properly check for gstvalue data types. This makes it possible to
 	set fields with fractions.
	* testsuite/test_caps.py:
	* testsuite/test_struct.py:
	Test for above modifications