Re: nnmail-split-fancy-match-partial-words

[Katsumi Yamaoka <yamaoka@jpl.org>]

>>>>> In <slpzq2t8.fsf@esemetz.metz.supelec.fr>
>>>>>	Fabrice Popineau wrote:

> Id' like to draw attention on the following point. I have a rather long
> nnmail-split-fancy list. I had this split rule :

> ("subject" "\\[ce\\]" foo.ce)

> that was not working and I had much trouble in finding why. In fact,
> that is because of interaction between
> nnmail-split-fancy-match-partial-words and the status of [] that do not
> belong to words in the default syntax table. While I'd like to retain
> the default nil value for matching partial words, I'd expect also that
> any regexp I put in the fancy split rule is taken for what it is.

> I ended up in crudely patching nnmail-split-it in nnmail.el but that's
> defintely not a clean answer to the problem. Maybe that's a limitation
> of regexp/syntax tables after all.

> -	(when nnmail-split-fancy-match-partial-words
> +	(when
> +	    (or nnmail-split-fancy-match-partial-words
> +		;; FP: if there are backets in the sequence, don't surround with <..>
> +		(string-match "\\[.*\\]" value))

Thank you for the patch, but it seems to be better to use
"\\\\\\[.*\\\\\\]" because "\\[.*\\]" matches "[0-9]", "[a-z]",
etc.  Does anyone notice other adverse effects?

> BTW, all occurences of set-file-modes should be replaced with
> gnus-set-file-modes for windows users'sake (at least under xemacs).

> -    (set-file-modes filename nnmail-default-file-modes)))
> +    (gnus-set-file-modes filename nnmail-default-file-modes)))

I think it should be applied anyway.  mail-source.el and
mm-decode.el also use set-file-modes.

> I'm using xemacs 21.5.b21 under native windows for what it worth.

I will install those changes if no one comments within a couple
of days.