Hello!

On Tue, Jan 29, 2013 at 06:49:46PM +0100, Peter Wu wrote:

[...]

> RFC: this patch only adds support for URL parameters. Before implementing the 
> actual clickable headers I would like to have some feedback. Do you actually
> want to add such a sorting feature to the standard autoindex module? Sorting
> has also been requested on http://forum.nginx.org/read.php?10,211728

Overrall I tend to think that it's bad idea to do autoindex more 
customizable.  I would rather like to see it producing XML and 
hence any customization being possible with XSLT filter.

[...]

> Parsing URL parameters is done in a separate function which is currently
> non-void. Since the return value is not used at the moment, it can also be 
> make
> void. Thoughts?

You may want to use ngx_http_arg() function instead of reinventing 
the wheel.

-- 
Maxim Dounin
http://nginx.com/support.html

_______________________________________________
nginx-devel mailing list
nginx-devel@nginx.org
http://mailman.nginx.org/mailman/listinfo/nginx-devel

Reply via email to