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 160318 - [ff[dec|enc]_mjpeg] should support image/jpeg
[ff[dec|enc]_mjpeg] should support image/jpeg
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-libav
git master
Other Linux
: Normal enhancement
: 0.8.3
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2004-12-03 10:27 UTC by Luca Ognibene
Modified: 2004-12-22 21:47 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Luca Ognibene 2004-12-03 10:27:58 UTC
gst-inspect ffdec_mjpeg
gst-inspect ffenc_mjpeg

PATCH: in gstffmpegcodecmap.c, case CODEC_ID_MJPEG:
add gst_caps_append (caps, GST_FF_VID_CAPS_NEW ("image/jpeg", NULL));
Comment 1 David Schleef 2004-12-04 02:03:49 UTC
JPEG is not Motion JPEG.

Do you have evidence that ffmpeg decodes actual JPEG images?
Comment 2 Luca Ognibene 2004-12-04 09:01:53 UTC
Yes, i've tried it. It works also better than jpegdec that fails on some images with "unsupported marker 
type" error.
You can try it with: gst-launch filesrc location="file.jpg" blocksize=999999 ! image/
jpeg,framerate=1.0 ! ffdec_jpeg ! ffcolorspace ! ximagesink
you can also go to: http://ffmpeg.sourceforge.net/ffmpeg-doc.html#SEC18
Comment 3 Ronald Bultje 2004-12-04 09:06:18 UTC
I think it supports it. Anyway, we're currently using image/jpeg everywhere in
qtdemux/avidemux, and the separation between video and image makes no sense in
practice (even though I'd love to) because most apps just decode plain JPEG
anyway, of which MJPEG is just a subset.

Let's just change it.

Those marker messages need to be hidden under GST_DEBUG, but then I first need
to find out how to wrap the libjpeg stdout messages.
Comment 4 Ronald Bultje 2004-12-16 20:26:37 UTC
Changed.