bug#5342: 23.1; mairix-widget-search always shows whole threads

[David Abrahams]

Please write in English if possible, because the Emacs maintainers
usually do not have translators to read other languages for them.

Your bug report will be posted to the bug-gnu-emacs@gnu.org mailing list,
and to the gnu.emacs.bug news group.

Please describe exactly what actions triggered the bug
and the precise symptoms of the bug:

`M-x mairix-widget-search'
search on body containing "foobar"
do NOT check "Show full threads"
finds all messages in threads that mention foobar in the body of any message.

Tracked it down to 

  (defun mairix-widget-send-query (widgets)
    "Send query from WIDGETS to mairix binary."
    (mairix-search
     (mairix-widget-make-query-from-widgets widgets)
     (if (widget-value (cadr (assoc "Threads" widgets)))
         t
       -1))
  ;    ^^---------this should be nil
    (kill-buffer mairix-customize-query-buffer))


If Emacs crashed, and you have the Emacs process in the gdb debugger,
please include the output from the following gdb commands:
    `bt full' and `xbacktrace'.
If you would like to further debug the crash, please read the file
/Applications/Emacs.app/Contents/Resources/etc/DEBUG for instructions.


In GNU Emacs 23.1.1 (i386-apple-darwin9.8.0, NS apple-appkit-949.54)
 of 2009-08-16 on black.local
Windowing system distributor `Apple', version 10.3.1038
configured using `configure  '--with-ns''

Important settings:
  value of $LC_ALL: nil
  value of $LC_COLLATE: nil
  value of $LC_CTYPE: nil
  value of $LC_MESSAGES: nil
  value of $LC_MONETARY: nil
  value of $LC_NUMERIC: nil
  value of $LC_TIME: nil
  value of $LANG: nil
  value of $XMODIFIERS: nil
  locale-coding-system: nil
  default-enable-multibyte-characters: t

Major mode: Emacs-Lisp

Minor modes in effect:
  show-paren-mode: t
  server-mode: t
  global-auto-revert-mode: t
  delete-selection-mode: t
  tooltip-mode: t
  mouse-wheel-mode: t
  menu-bar-mode: t
  file-name-shadow-mode: t
  global-font-lock-mode: t
  font-lock-mode: t
  global-auto-composition-mode: t
  auto-composition-mode: t
  auto-encryption-mode: t
  auto-compression-mode: t
  line-number-mode: t
  transient-mark-mode: t

Recent input:
C-n C-n C-n C-n C-n C-n C-n C-n <return> <tab> m a 
i r i x C-n C-n C-n C-n C-n C-n C-n C-n C-n C-n C-n 
C-n C-n C-a C-f <return> C-n C-n C-n C-p C-e C-b C-f 
C-x C-e C-n C-n C-b C-p C-p C-p C-p C-p C-n M-x f i 
n d - f u n <tab> <return> <return> C-p C-n C-n C-n 
C-. C-. C-. C-. C-. C-n C-n C-p C-p C-p C-p C-s t h 
r e a d s C-n C-n C-n C-n C-n C-n C-n C-n C-n C-n C-n 
C-, C-, C-n C-p C-p C-p C-n C-n C-n C-p C-p C-p M-x 
M-p <return> <return> C-n C-n C-n C-n C-n C-p C-p C-p 
C-p C-n C-n C-p C-p C-p C-p C-n C-n C-s t h r e a d 
s C-x C-b C-x C-b C-p C-p C-p C-p C-p C-p C-p C-p C-p 
C-p C-p C-p C-p C-p C-n C-n C-n C-n C-n C-n C-n C-n 
C-n C-n C-n C-n C-n C-n C-n C-p C-p C-p C-p C-p C-p 
C-p C-p C-p C-p C-p C-p C-p C-p C-n C-n C-n C-n C-n 
C-n C-n C-n C-n C-n C-n C-n C-n C-p C-p C-p C-p C-p 
C-p C-p C-p C-p C-p C-p C-p C-p C-p C-p C-p C-p C-p 
C-p C-p C-p C-p C-p C-p C-p C-p C-p C-p C-p C-p C-p 
C-p C-p C-p C-p C-p C-p C-p C-p C-p C-p C-p C-p C-p 
C-p C-p C-p C-p C-p C-p C-p C-n C-n C-n C-n C-s - 1 
M-< C-n C-n C-n C-n C-n C-n C-p C-p C-p C-e C-n C-e 
C-, C-, C-, C-, C-, C-SPC C-e M-w C-n C-n C-n C-n C-n 
C-n C-n C-n C-n C-n C-n C-n M-x r e p o r t - e m <tab> 
<return>

Recent messages:
Invalid image size (see `max-image-size') [25 times]
Mark set
Invalid image size (see `max-image-size') [97 times]
Mark saved where search started
Invalid image size (see `max-image-size') [50 times]
Mark saved where search started
Mark set
Invalid image size (see `max-image-size')
Mark set
Invalid image size (see `max-image-size') [24 times]
-- 
Dave Abrahams           Meet me at BoostCon: http://www.boostcon.com
BoostPro Computing
http://www.boostpro.com