[vlc-devel] [PATCH] Implemented libjpeg decoder module

Maxim Bublis b at codemonkey.ru
Sat Dec 14 04:41:49 CET 2013


> No, but an encoder, so doing snapshots in jpeg, would not call
> libavcodec, which is slow and has low encoding quality (see #5943)


Oh, ok, I understand now, have seen avcodec as an example.
I think it would be a separate patch. I'll look at it in the nearest time.


> > > > diff --git a/src/misc/fourcc.c b/src/misc/fourcc.c
> > > > index 8cd821a..04a4c71 100644
> > > > --- a/src/misc/fourcc.c
> > > > +++ b/src/misc/fourcc.c
> > > > @@ -351,8 +351,6 @@ static const staticentry_t p_list_video[] = {
> > > >          A("mjpg"),
> > > >          A("mJPG"),
> > > >          A("mjpa"),
> > > > -        A("jpeg"),
> > > > -        A("JPEG"),
> > > >          A("JFIF"),
> > > >          A("JPGL"),
> > > >          A("LJPG"),
> > >
> > > Hmm, why?
> > >
> >
> > Image demuxer detects JPEG images incorrectly as Motion JPEG videos.
> > This lines dropped from Motion JPEG part only.
>
> Well, some video with mjpeg might use those FOURCC, so you would break
> them.


As far as I understand Lookup function code in fourcc.c it wouldn't break
anything, just will fix description for both 'jpeg' and 'JPEG' fourcc's as
they appear under JPEG image section. Am I wrong?

-- 
Maxim Bublis
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20131214/5c2e63fd/attachment.html>


More information about the vlc-devel mailing list