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 644416 - [encodebin] Cannot be reused
[encodebin] Cannot be reused
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-base
unspecified
Other All
: Normal normal
: 0.10.33
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2011-03-10 17:25 UTC by Thiago Sousa Santos
Modified: 2011-03-11 00:20 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
tests: encodebin: Add reuse test case (2.48 KB, patch)
2011-03-10 17:25 UTC, Thiago Sousa Santos
none Details | Review
encodebin: Tear down old profiles when setting new ones (1.11 KB, patch)
2011-03-10 17:40 UTC, Thiago Sousa Santos
committed Details | Review

Description Thiago Sousa Santos 2011-03-10 17:25:00 UTC
Taking encodebin into paused and then back to null to change its profile
fails.

I'm attaching a test case.
Comment 1 Thiago Sousa Santos 2011-03-10 17:25:02 UTC
Created attachment 183075 [details] [review]
tests: encodebin: Add reuse test case

Adds a test case to check if encodebin can be reused
Comment 2 Edward Hervey 2011-03-10 17:37:41 UTC
In NULL/READY it should be able to change the profile, and if a profile was previously present it should tear it down.

In PAUSED/PLAYING it should refuse setting a new profile.

That test makes sense.
Comment 3 Thiago Sousa Santos 2011-03-10 17:40:21 UTC
Created attachment 183078 [details] [review]
encodebin: Tear down old profiles when setting new ones

In NULL/READY, we should be able to switch profiles on encodebin,
this patch makes it tear down old profiles when new ones are set
if in NULL/READY states
Comment 4 Thiago Sousa Santos 2011-03-11 00:20:22 UTC
commit 40eaac6191b4a3ab58a5d134e6b2ad015ec5b19d
Author: Thiago Santos <thiago.sousa.santos@collabora.co.uk>
Date:   Thu Mar 10 14:22:38 2011 -0300

    tests: encodebin: Add reuse test case
    
    Adds a test case to check if encodebin can be reused
    
    https://bugzilla.gnome.org/show_bug.cgi?id=644416

commit 4ee738be590348912ae3af607711af8b4550ad3f
Author: Thiago Santos <thiago.sousa.santos@collabora.co.uk>
Date:   Thu Mar 10 14:38:47 2011 -0300

    encodebin: Tear down old profiles when setting new ones
    
    In NULL/READY, we should be able to switch profiles on encodebin,
    this patch makes it tear down old profiles when new ones are set
    if in NULL/READY states
    
    https://bugzilla.gnome.org/show_bug.cgi?id=644416