unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
* bug#6613: 23.2; bogus let bound var in gnus-delay.el
@ 2010-07-11 21:07 Leo
  2010-10-18 20:48 ` Lars Magne Ingebrigtsen
  0 siblings, 1 reply; 2+ messages in thread
From: Leo @ 2010-07-11 21:07 UTC (permalink / raw)
  To: 6613

In the function `gnus-delay-article' in gnus-delay.el,

  (let ((group (format "nndraft:%s" gnus-delay-group)))
      (gnus-agent-queue-setup gnus-delay-group))

The let-bound var `group' is not used and in fact gnus-agent-queue-setup
takes care of formatting the group name already. So I think it should be
simplified to

  (gnus-agent-queue-setup gnus-delay-group)

Leo





^ permalink raw reply	[flat|nested] 2+ messages in thread

* bug#6613: 23.2; bogus let bound var in gnus-delay.el
  2010-07-11 21:07 bug#6613: 23.2; bogus let bound var in gnus-delay.el Leo
@ 2010-10-18 20:48 ` Lars Magne Ingebrigtsen
  0 siblings, 0 replies; 2+ messages in thread
From: Lars Magne Ingebrigtsen @ 2010-10-18 20:48 UTC (permalink / raw)
  To: Leo; +Cc: 6613-close

Leo <sdl.web@gmail.com> writes:

> In the function `gnus-delay-article' in gnus-delay.el,
>
>   (let ((group (format "nndraft:%s" gnus-delay-group)))
>       (gnus-agent-queue-setup gnus-delay-group))
>
> The let-bound var `group' is not used and in fact gnus-agent-queue-setup
> takes care of formatting the group name already. So I think it should be
> simplified to
>
>   (gnus-agent-queue-setup gnus-delay-group)

This has now been fixed in git Gnus.

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





^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2010-10-18 20:48 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-07-11 21:07 bug#6613: 23.2; bogus let bound var in gnus-delay.el Leo
2010-10-18 20:48 ` Lars Magne Ingebrigtsen

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).