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 582081 - ffenc_mpeg4 should be exposed as ffenc_divx5, ffenc_divx4 and ffenc_mpeg4
ffenc_mpeg4 should be exposed as ffenc_divx5, ffenc_divx4 and ffenc_mpeg4
Status: RESOLVED INVALID
Product: GStreamer
Classification: Platform
Component: gst-libav
0.10.x
Other Linux
: Normal enhancement
: git master
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2009-05-10 16:24 UTC by Christian Fredrik Kalager Schaller
Modified: 2012-11-20 09:19 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Christian Fredrik Kalager Schaller 2009-05-10 16:24:51 UTC
So as the only encoder I found so far ffenc_mpeg4 got two SRC pads.

One is video/mpeg,mpegversion=4 and the other is video/x-divx,divxversion=5.

So according to the ffmpeg FAQ http://ffmpeg.mplayerhq.hu/faq.html#SEC23 there difference between these two (and divxversion=4) which you can also set on the ffmpeg level is the FOURCC value.

Having to take this situation into account would make my code quite a bit more complex as I can no longer iterate on element to create my dictionary between caps and elements, with caps as the unique value, but have to instead iterate on caps and at the same time finding back to the original element which had that caps.

My suggestion is that we do here what we did for qtmux, which is to have one virtual element for each caps type.
Comment 1 Christian Fredrik Kalager Schaller 2009-05-10 19:24:42 UTC
if switching to 3 virtual elements I would ask that a video/x-divx,divxversion=4 SRC caps is added so I can get the DIVX fourcc value to put into the AVI file
Comment 2 Tim-Philipp Müller 2010-08-04 12:27:43 UTC
These aren't source pads, just different possible output caps on the source template.

Exposing these as different elements sounds like a good idea to me.
Comment 3 Christian Fredrik Kalager Schaller 2012-11-20 09:19:51 UTC
ok, I think in 1.0 we did away with the divx caps, so closing this bug.