[vlc-devel] [PATCH] codec: jpeg encoder implemented
jb at videolan.org
Fri Jan 24 17:10:22 CET 2014
On 24 Jan, Maxim Bublis wrote :
> No quality set ?
> I guess, configuration option should be added for jpeg screenshots, but
> currently no one exists.
> I've hardcoded 100% quality for now, but I'll look how to add configuration
What's the size for 95% ?
> > And I wonder whether we should, or not disable-sout for images (because
> > it might be nice for screenshots).
> I've looked at avcodec code, as you know it is currently used for jpeg
> screenshots, it uses the same #ifdef for its encoder.
> Maybe it should be removed in case of jpeg encoder.
Well, avcodec is big. This might not be.
> > Why removing all this code?
> I've found this code useless, as on any error libjpeg always setjmp's, and
> that code never executes.
Then, it needs a different commit.
With my kindest regards,
http://www.jbkempf.com/ - +33 672 704 734
Sent from my Electronic Device
More information about the vlc-devel