GNU bug report logs -
#42977
28.0.50; New gnus-dbus library providing (optional) D-Bus->Gnus integration
Previous Next
Reported by: Eric Abrahamsen <eric <at> ericabrahamsen.net>
Date: Fri, 21 Aug 2020 20:44:02 UTC
Severity: normal
Found in version 28.0.50
Done: Eric Abrahamsen <eric <at> ericabrahamsen.net>
Bug is archived. No further changes may be made.
Full log
Message #26 received at 42977 <at> debbugs.gnu.org (full text, mbox):
On 08/25/20 21:21 PM, Lars Ingebrigtsen wrote:
> Eric Abrahamsen <eric <at> ericabrahamsen.net> writes:
>
>> Okay, there it goes. This was made a tiny bit more difficult to test by
>> the fact that the 'closed status (in `gnus-opened-servers') is set at
>> various calling sites, and not centrally in `gnus-close-server'.
>> `gnus-open-server' sets the 'open status, it seems like
>> `gnus-close-server' should do the equivalent. I could take it out of
>> `gnus-group-suspend', and look at whether `gnus-server-set-status' was
>> really necessary.
>
> I think the use case is stuff like this:
>
> (defun gnus-agent-toggle-group-plugged (group)
> "Toggle the status of the server of the current group."
> (interactive (list (gnus-group-group-name)))
> (let* ((method (gnus-find-method-for-group group))
> (status (cadr (assoc method gnus-opened-servers))))
> (if (eq status 'offline)
> (gnus-server-set-status method 'closed)
> (gnus-close-server method)
> (gnus-server-set-status method 'offline))
>
> Where we close the server (i.e., the backends close the network
> connections), but don't set the status to 'closed, because it's still
> open on the Gnus side.
Okay, I see. But even given that, leaving `gnus-server-set-status'
alone, it might be fine to have `gnus-close-server' set a 'closed status
-- the server will just re-set it afterwards.
Alternately, we could just have `gnus-close-all-servers' set the status,
too.
This bug report was last modified 4 years and 271 days ago.
Previous Next
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997,2003 nCipher Corporation Ltd,
1994-97 Ian Jackson.