gnus-string-equal broken by using compare-strings

gnus-string-equal broken by using compare-strings

[Torsten Hilbrich]

Hello

I just update gnus from CVS and got the following error when
displaying any article:

Debugger entered--Lisp error: (wrong-type-argument stringp nil)
  compare-strings(nil nil nil "local.debian.user-german" nil nil t)
  gnus-string-equal(nil "local.debian.user-german")
  article-hide-boring-headers()
  apply(article-hide-boring-headers nil)
  (if interactive (call-interactively (quote article-hide-boring-headers)) (apply (quote article-hide-boring-headers) args))
  (save-excursion (set-buffer gnus-article-buffer) (if interactive (call-interactively ...) (apply ... args)))
  gnus-article-hide-boring-headers()
  gnus-treat-article(head)
  gnus-display-mime()
  gnus-article-prepare-display()
  gnus-article-prepare(46120 nil)
  gnus-summary-display-article(46120 nil)
  gnus-summary-select-article(nil nil pseudo)
  gnus-summary-scroll-up(1)
  call-interactively(gnus-summary-scroll-up)

I debugged a little bit and found out that the error happens here:

Function: article-hide-boring-headers, line 1664:

	     ((eq elem 'followup-to)
	      (when (gnus-string-equal
		     (message-fetch-field "followup-to")
		     (message-fetch-field "newsgroups"))
		(gnus-article-hide-header "followup-to")))

(message-fetch-field "followup-to") returns nil and gnus-string-equal
bails out because it is defined as:

    (defun gnus-string-equal (x y)
      "Like `string-equal', except it compares case-insensitively."
      (eq t (compare-strings x nil nil y nil nil t)))

compare-strings doesn't like its x/y arguments to be nil.

I checked against the older version and there the compare-string
function is not used at all, so the problem seems to be this change.

        Torsten

Re: gnus-string-equal broken by using compare-strings

[Lars Magne Ingebrigtsen]

Torsten Hilbrich <gnus@myrkr.in-berlin.de> writes:

> Debugger entered--Lisp error: (wrong-type-argument stringp nil)
>   compare-strings(nil nil nil "local.debian.user-german" nil nil t)

This has apparently been fixed by now.

-- 
(domestic pets only, the antidote for overdose, milk.)
   larsi@gnus.org * Lars Magne Ingebrigtsen