Should I send patch here? - xslt_enable "only enable xslt transform for certain user agent"

[tOmasEn]

Maybe not many people using xslt as main web layout. but I do.

This patch is similar to gzip_disable.  xslt_enable only enable xslt
transform for certain user agent.

e.g.
xslt_enable "Googlebot"
will only enable xslt transform output for google's crawler.

And, I'm also planing some other improve on xslt module. Like: when xslt
transform failed, output original xml file instead of 500 server error; Skip
xslt transform if client send specified HTTP header like "No-XSLT", etc.
Should I talk to somebody and discuss the architect?

-- 
Tomasen
http://twitter.com/ShooterPlayer
http://t.sina.com.cn/Tomasen
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://nginx.org/pipermail/nginx/attachments/20100127/05272373/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: xslt_enable.patch
Type: application/octet-stream
Size: 4997 bytes
Desc: not available
URL: <http://nginx.org/pipermail/nginx/attachments/20100127/05272373/attachment.obj>