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 752493 - harness: Improve detection of element type
harness: Improve detection of element type
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gstreamer (core)
unspecified
Other All
: Normal normal
: 1.5.90
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2015-07-16 14:41 UTC by Stian Selnes (stianse)
Modified: 2015-08-16 13:40 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
harness: Improve detection of element type (9.16 KB, patch)
2015-07-16 14:41 UTC, Stian Selnes (stianse)
committed Details | Review

Description Stian Selnes (stianse) 2015-07-16 14:41:51 UTC
The attached patch improves how GstHarness will detect how to setup an element, particularly a bin.
Comment 1 Stian Selnes (stianse) 2015-07-16 14:41:55 UTC
Created attachment 307560 [details] [review]
harness: Improve detection of element type

The element flag does not indicate wether a bin should be tested as a
source or as a sink, eg. a bin with the sink flag may still have a
source pad and a bin with the source flag may have a sink pad. In this
case it is better to determine the element type by looking at the
available pads and pad templates.

Also rename srcpad and sinkpad where it actually represents
element_srcpad_name and element_sinkpad_name.
Comment 2 Tim-Philipp Müller 2015-07-16 15:26:35 UTC
This didn't apply because of the header changes I made, but I fixed it up:

commit 8597284d0be697bffbdb9f31a0f4e6576bc0a73f
Author: Stian Selnes <stian@pexip.com>
Date:   Mon Jul 13 11:03:13 2015 +0200

    harness: Improve detection of element type
    
    The element flag does not indicate wether a bin should be tested as a
    source or as a sink, eg. a bin with the sink flag may still have a
    source pad and a bin with the source flag may have a sink pad. In this
    case it is better to determine the element type by looking at the
    available pads and pad templates.
    
    Also rename srcpad and sinkpad where it actually represents
    element_srcpad_name and element_sinkpad_name.
    
    https://bugzilla.gnome.org/show_bug.cgi?id=752493


But please update/rebase before making more patches against the header :)