<div dir="ltr"><br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">No, but an encoder, so doing snapshots in jpeg, would not call<br>

libavcodec, which is slow and has low encoding quality (see #5943)</blockquote><div><br></div><div>Oh, ok, I understand now, have seen avcodec as an example.</div><div>I think it would be a separate patch. I'll look at it in the nearest time.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">
> > > diff --git a/src/misc/fourcc.c b/src/misc/fourcc.c<br>
> > > index 8cd821a..04a4c71 100644<br>
> > > --- a/src/misc/fourcc.c<br>
> > > +++ b/src/misc/fourcc.c<br>
> > > @@ -351,8 +351,6 @@ static const staticentry_t p_list_video[] = {<br>
> > >          A("mjpg"),<br>
> > >          A("mJPG"),<br>
> > >          A("mjpa"),<br>
> > > -        A("jpeg"),<br>
> > > -        A("JPEG"),<br>
> > >          A("JFIF"),<br>
> > >          A("JPGL"),<br>
> > >          A("LJPG"),<br>
> ><br>
> > Hmm, why?<br>
> ><br>
><br>
> Image demuxer detects JPEG images incorrectly as Motion JPEG videos.<br>
> This lines dropped from Motion JPEG part only.<br>
<br>
</div>Well, some video with mjpeg might use those FOURCC, so you would break<br>
them.</blockquote><div><br></div><div>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?</div>
</div><div><br></div>-- <br>Maxim Bublis
</div></div>