bug#19026: 24.4.50; rmail-summary-by-topic

bug#19026: 24.4.50; rmail-summary-by-topic

[Richard Stallman]

In rmail-summary-by-topic, if (rmail-simplify-subject) returns a
string with a comma, that is taken as a comma-separated list of
different things to search for.  Thus, it finds messages whose
subject is just a part of this message's subject.



In GNU Emacs 24.4.50.1 (i686-pc-linux-gnu, GTK+ Version 2.24.10)
 of 2014-07-18 on rms
Repository revision: 117549 dmantipov@yandex.ru-20140718110437-w0451dqdaa977r4w
System Description:	Trisquel GNU/Linux 6.0.1, Toutatis

Configured using:
 `configure 'CFLAGS=-g -O0''

Configured features:
XPM JPEG TIFF GIF PNG RSVG SOUND GPM DBUS GCONF GSETTINGS NOTIFY
LIBXML2 FREETYPE M17N_FLT LIBOTF XFT ZLIB

Important settings:
  value of $LANG: en_US.UTF-8
  locale-coding-system: utf-8-unix

Major mode: Emacs-Lisp

Minor modes in effect:
  diff-auto-refine-mode: t
  shell-dirtrack-mode: t
  gpm-mouse-mode: t
  tooltip-mode: t
  mouse-wheel-mode: t
  tool-bar-mode: t
  menu-bar-mode: t
  file-name-shadow-mode: t
  global-font-lock-mode: t
  font-lock-mode: t
  auto-composition-mode: t
  auto-encryption-mode: t
  auto-compression-mode: t
  line-number-mode: t
  abbrev-mode: t


-- 
Dr Richard Stallman
President, Free Software Foundation
51 Franklin St
Boston MA 02110
USA
www.fsf.org  www.gnu.org
Skype: No way! That's nonfree (freedom-denying) software.
  Use Ekiga or an ordinary phone call.

bug#19026: 24.4.50; rmail-summary-by-topic

[Glenn Morris]

Richard Stallman wrote:

> In rmail-summary-by-topic, if (rmail-simplify-subject) returns a
> string with a comma, that is taken as a comma-separated list of
> different things to search for.  Thus, it finds messages whose
> subject is just a part of this message's subject.

I wrote a FIXME comment about this 5 years ago:
  
  ;; FIXME "a string of regexps separated by commas" makes no sense because:
  ;;  i) it's pointless (you can just use \\|)
  ;; ii) it's broken (you can't specify a literal comma)
  ;; rmail-summary-by-topic and rmail-summary-by-senders have the same issue.

I suggest just dumping this "separated by commas" idea.
Ie, you would have to write "foo\\|bar" like in any other regexp.

bug#19026: 24.4.50; rmail-summary-by-topic

[Glenn Morris]

Version: 25.1

Glenn Morris wrote:

> I suggest just dumping this "separated by commas" idea.
> Ie, you would have to write "foo\\|bar" like in any other regexp.

Installed in the absence of any comments.