Thanks for the patch!
On 02/05/2014 02:57 PM, Nick Kew wrote:
The hesitation is because I've been wanting to review the
patch before committing, and round tuits are in woefully
short supply. So I'm attaching it here. I'll take any feedback
from you or other users as a substitute for my own review,
and commit if it works for you without glitches.
Minor glitch: the patch doesn't compile because it uses the unknown
variable cfg in xml2enc_ffunc(). Otherwise it works as advertised.
My wishlist:
* Make the configuration option as powerful as the compiled in fallback
so that you can configure eg. "contains xml". But how would you do that?
Support regular expressions?
* Provide a configuration option to blacklist content types so that you
can use the defaults that are compiled in but exclude specific types
from processing (this is how I work around the Sharepoint problem, I
simply exclude content type "multipart/related").
