<br><div class="gmail_extra"><br><br><div class="gmail_quote">On 5 December 2012 18:52, Maxim Dounin <span dir="ltr"><<a href="mailto:mdounin@mdounin.ru" target="_blank">mdounin@mdounin.ru</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

Hello!<br>
<div class="im"><br>
On Tue, Dec 04, 2012 at 06:58:17PM +0400, ivan babrou wrote:<br>
<br>
> ><br>
> > Additionally, looking at the code I tend to think it's incorrect.<br>
> > That is, it's pattern you've followed is incorrect, not your code.<br>
> > E.g. the following config will result in $arg_q incorrectly used<br>
> > for quality in /image/, instead of "80" explicitly set:<br>
> >     image_filter_jpeg_quality $arg_q;<br>
> >     location /image/ {<br>
> >         image_filter crop $arg_w $arg_h;<br>
> >         image_filter_jpeg_quality 80;<br>
> >     }<br>
> > This needs fixing.<br>
><br>
><br>
> As I see other config variables are buggy too. Should I fix them too?<br>
<br>
</div>It would be fine to produce a patch which fixes existing config<br>
variables, and the patch to add crop offsets on top of it.</blockquote><div><br></div><div>Can you give me some good examples to see "good practices" in nginx configuration handling?</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div class="im">
> Anyway, this is latest version of my patch where I fixed other issues:<br>
<br>
</div>I don't see at least some of previously expressed points<br>
addressed, so I assume you are still working on it.<br>
<br>
Please also see other point rightfully noted by Ruslan Khusnullin -<br>
you can't use ngx_strcmp() in ngx_http_image_filter_get_value() as<br>
it's used not only on null-terminated strings.<br>
<div class="HOEnZb"><div class="h5"><br>
--<br>
Maxim Dounin<br>
<a href="http://nginx.com/support.html" target="_blank">http://nginx.com/support.html</a><br>
<br>
_______________________________________________<br>
nginx-devel mailing list<br>
<a href="mailto:nginx-devel@nginx.org">nginx-devel@nginx.org</a><br>
<a href="http://mailman.nginx.org/mailman/listinfo/nginx-devel" target="_blank">http://mailman.nginx.org/mailman/listinfo/nginx-devel</a><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br>Regards, Ian Babrou<br><a href="http://bobrik.name" target="_blank">http://bobrik.name</a> <a href="http://twitter.com/ibobrik" target="_blank">http://twitter.com/ibobrik</a> skype:i.babrou<br>


</div>