<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<div><div class="h5"><span style="color:rgb(34,34,34)">So, both are the same?</span></div></div></blockquote><div><br></div><div>Yup, do you have any ideas how to improve that?</div><div>My initial attempt was to create struct jpeg_sys_t and then just write something like:</div>
<div><br></div><div>typedef struct jpeg_sys_t encoder_sys_t;</div><div>typedef struct jpeg_sys_t decoder_sys_t;</div><div><br></div><div>but it leads to typedef redefinition with different types for encoder_sys_t/decoder_sys_t.</div>
<div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">No quality set ?<br></blockquote><div><br></div>
<div>I guess, configuration option should be added for jpeg screenshots, but currently no one exists.</div><div>I've hardcoded 100% quality for now, but I'll look how to add configuration options.</div><div> </div>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
And I wonder whether we should, or not disable-sout for images (because<br>
it might be nice for screenshots).</blockquote><div><br></div><div>I've looked at avcodec code, as you know it is currently used for jpeg screenshots, it uses the same #ifdef for its encoder.</div><div>Maybe it should be removed in case of jpeg encoder.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">Why removing all this code?</blockquote><div><br>
</div><div>I've found this code useless, as on any error libjpeg always setjmp's, and that code never executes.</div><div>My initial work on this module was based on png module, I guess libpng works slight differently in this case.</div>
<div><br></div></div>-- <br>Maxim Bublis
</div></div>