Re: Handle empty regexp in `rmail-nonignored-headers'

[Stefan Monnier <monnier@iro.umontreal.ca>]

> The default value for `rmail-nonignored-headers' is "^x-spam-status:".
> If the user wants to remove this header from the list of non-ignored
> headers, she will set `rmail-nonignored-headers' to the empty string.
> This will end up in *not* ignoring any header, which is wrong.

> This simple patch fixed this -- can I commit it?

I'd rather allow rmail-nonignored-headers to be nil (to mean "don't
unignore any headers") and let "" have its standard meaning of "match
everything/everywhere".


        Stefan


> --- rmail.el.~1.447.~	2008-02-06 23:54:33.000000000 +0000
> +++ rmail.el	2008-02-10 16:30:56.000000000 +0000
> @@ -2329,7 +2329,8 @@
>  	  (while (and ignored-headers
>  		      (re-search-forward ignored-headers nil t))
>  	    (beginning-of-line)
> -	    (if (looking-at rmail-nonignored-headers)
> +	    (if (and (not (equal rmail-nonignored-headers ""))
> +		     (looking-at rmail-nonignored-headers))
>  		(forward-line 1)
>  	      (delete-region (point)
>  			     (save-excursion

> -- 
> Bastien