GNU bug report logs -
#30235
[patch] undo-tree byte-compiler warnings
Previous Next
Reported by: Alex Branham <alex.branham <at> gmail.com>
Date: Tue, 23 Jan 2018 23:21:02 UTC
Severity: minor
Tags: patch, wontfix
Merged with 24354
Done: Lars Ingebrigtsen <larsi <at> gnus.org>
Bug is archived. No further changes may be made.
Full log
View this message in rfc822 format
Lars Ingebrigtsen <larsi <at> gnus.org> writes:
> Alex Branham <alex.branham <at> gmail.com> writes:
>
>> I'm not sure if this is the right place to submit patches for ELPA
>> packages, but I couldn't find anywhere else to go. Let me know if this
>> should go some place else.
>>
>> This patch silences several byte compiler warnings, mostly having to do
>> with unused lexical variables in loops.
>
> The unused dolist fixes look OK, but they could also be rewritten as
>
> (cl-loop repeat ...
> do ...)
>
> to avoid the binding altogether.
There's really no need to completely rewrite all those dotimes in terms
of cl-loop. Writing (dotimes (_ ...) ...) is quite common and fine.
> The other changes (time-subtract/subtract-time) I'm not so sure about,
> because ELPA things are supposed to work across a large number of Emacs
> versions, and time-subtract is newer than subtract-time.
Sure, but subtract-time has been called time-subtract for almost two
decades now[1].
[1: 74fcda73dd]: Add autoload cookies. Many doc fixes.
2002-01-27 23:30:29 +0000
https://git.savannah.gnu.org/cgit/emacs.git/commit/?id=74fcda73dd8743f66256aa5fdaff6260dc356c54
> The same with the other obsolete functions -- you're bound to get those
> warnings in ELPA packages.
Sure, but often a compatibility shim can go a long way, and in this case
the question is why was undo-elt-crosses-region obsoleted as part of
bug#17235, and what should undo-tree.el do in future Emacs versions.
Maybe Barry, Toby, or Stefan should be CCed in this or a new discussion
about it, seeing as Jonas' concerns were never addressed, AFAICT.
Thanks,
--
Basil
This bug report was last modified 5 years and 281 days ago.
Previous Next
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997,2003 nCipher Corporation Ltd,
1994-97 Ian Jackson.