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 346563 - gstgnomevfssrc doesn't run typefind
gstgnomevfssrc doesn't run typefind
Status: RESOLVED DUPLICATE of bug 341524
Product: GStreamer
Classification: Platform
Component: gst-plugins-base
git master
Other Linux
: Normal normal
: git master
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2006-07-04 16:21 UTC by Lutz Mueller
Modified: 2006-07-05 06:42 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Patch to add typefind to gstgnomevfssrc.c (1.07 KB, patch)
2006-07-04 16:22 UTC, Lutz Mueller
reviewed Details | Review

Description Lutz Mueller 2006-07-04 16:21:55 UTC
gstgnomevfssrc doesn't run typefind on the first buffer. If the element gets added to the pipeline dynamically (i.e. by decodebin), further decoding will fail because the caps on the source pad will remain GST_CAPS_ANY.
Comment 1 Lutz Mueller 2006-07-04 16:22:48 UTC
Created attachment 68357 [details] [review]
Patch to add typefind to gstgnomevfssrc.c
Comment 2 Tim-Philipp Müller 2006-07-04 16:39:32 UTC
Yet it does not fail in practice with playbin/decodebin, since decodebin will automatically plug a typefind element.

_If_ gnomevfssrc is supposed to do typefinding, it

 a) should do so properly if pull-mode is supported (with random access)
 b) it should probably work similarly to filesrc, which has a
    "typefind" boolean property which is disabled by default

However, personally I don't really think this is something that needs fixing at this point. 

And in any case, typefinding a single buffer isn't really enough/possible in many cases, especially with non-local sources where you might get very small buffers. The typefind element takes care of that (and of proper error handling in the EOS-before-we-could-find-a-type case etc.).


I am aware of your bz2dec issue and I know I asked you to change it to do typefinding, so admittedly there's an inconsistency, but my answer to that would be that "sources are special" :)

In 0.11/0.12 we might want to do something better here (like application/x-typefindme caps or something like that).

Comment 3 Lutz Mueller 2006-07-05 06:42:26 UTC
I agree that we should do typefind outside of the gnomevfs element. I just uploaded a patch for gstdecodebin that does typefind for each element that has GST_CAPS_ANY on its source pad: http://bugzilla.gnome.org/show_bug.cgi?id=341524.

*** This bug has been marked as a duplicate of 341524 ***