[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

bug#34973: 27.0.50; gnus-summary-move-article fails due to remhash

From: Deus Max
Subject: bug#34973: 27.0.50; gnus-summary-move-article fails due to remhash
Date: Mon, 25 Mar 2019 02:09:02 +0200
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/27.0.50 (gnu/linux)

On Sun, Mar 24 2019, Basil L. Contovounesios wrote:

> tags 34973 patch
> quit
> Deus Max <address@hidden> writes:
>> Reading the manual
>> (https://www.gnu.org/software/emacs/manual/html_node/elisp/Hash-Access.html#Hash-Access)
>> on remhash, it says: "In Emacs Lisp, remhash always returns nil.", so I
>> modified the calling function gnus-dup-unsuppress-article in gnus-dup.el, as 
>> follows:
>> 157c157,159
>> <       (remhash id gnus-dup-hashtb))))
>> ---
>>>       (if (hash-table-p gnus-dup-hashtb)
>>>           (remhash id gnus-dup-hashtb)
>>>         nil))))
>> This patch seems to have fixed the problem and now
>> gnus-summary-move-article works fine. !
>> the value of gnus-dup-hashtb was nil, for the problem runs.
> The return value of gnus-dup-unsuppress-article doesn't matter, only
> that remhash not be passed a nil gnus-dup-hashtb.
> In other words, gnus-dup-unsuppress-article should either not be called
> or should handle the case when it is called before gnus-dup-open has
> initialised gnus-dup-hashtb.
> I attach a patch which follows the apparent gnus-sum.el convention of
> guarding calls to gnus-dup.el functions based on the value of
> gnus-suppress-duplicates.
Of course, much better.
Thank you.

> This fixes the immediate issue at hand, but I think there may still be a
> problem when gnus-suppress-duplicates is enabled at a later time.  I
> think that deserves a separate bug report, though.
> Thanks,

reply via email to

[Prev in Thread] Current Thread [Next in Thread]