[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#34974: 27.0.50; Moving article error with duplicate suppression disa
Basil L. Contovounesios
bug#34974: 27.0.50; Moving article error with duplicate suppression disabled
Sun, 24 Mar 2019 17:08:16 +0000
Gnus/5.13 (Gnus v5.13) Emacs/27.0.50 (gnu/linux)
Eric Abrahamsen <address@hidden> writes:
> "Basil L. Contovounesios" <address@hidden> writes:
>> With gnus-suppress-duplicates left at its default value of nil, trying
>> to move an article with 'B m <group>' gives me the following backtrace:
>> Debugger entered--Lisp error: (wrong-type-argument hash-table-p nil)
>> remhash("<redacted-message-id>" nil)
>> funcall-interactively(gnus-summary-move-article nil)
>> call-interactively(gnus-summary-move-article nil nil)
>> I'm no expert, and I haven't tried reproducing this with a minimal
>> config, but I think gnus-summary-move-article should not call
>> gnus-dup-unsuppress-article when gnus-suppress-duplicates is nil, right?
>> This issue seems to have been uncovered by the switch to hash-tables in
>> bug#33653. Previously, gnus-dup-unsuppress-article called unintern,
>> which would not complain when its second argument gnus-dup-hashtb was
>> nil, even though it probably should have.
>> Patch to follow.
> It looks like the only other call to `gnus-dup-unsuppress-articles' is
> wrapped in a check for `gnus-suppress-duplicates' -- I suppose it would
> be enough just to wrap this one as well?
That is what the proposed patch does. Is it okay to push this to
As I mention there, though, I'm not sure such a guard is sufficient in
the case that gnus-suppress-duplicates is enabled at a later stage, but
I will investigate and report this as a separate bug if needed.