GNU bug report logs - #30235
[patch] undo-tree byte-compiler warnings

Previous Next

Package: emacs;

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

From: Toby Cubitt <toby-dated-1570017117.01e866 <at> dr-qubit.org>
To: Lars Ingebrigtsen <larsi <at> gnus.org>, Alex Branham <alex.branham <at> gmail.com>
Cc: 30235 <at> debbugs.gnu.org, toby-undo-tree <at> dr-qubit.org
Subject: bug#30235: [patch] undo-tree byte-compiler warnings
Date: Wed, 18 Sep 2019 12:51:46 +0100
Hi Alex,

Not opposed to this at all! (I forgot what i said previously, but if I was then it was before I knew of the _ method.) Indeed, I think I already made exactly this change a while back in the undo-tree development repo, but maybe forgot to transfer this into ELPA.

Undo-tree went into ELPA before there was support for separate per-package git repos in ELPA, which is why this isn't automatic. It makes upstreaming changes from my development repo(s) to ELPA that bit more complicated than a simple git push. I'm very busy with real life these days, so this inevitably gets forgotten/postponed. Maybe this is a good opportunity to shift all my ELPA packages over to per-package git repos, so I can just make the ELPA repos upstreams of my package development repos?

Best wishes,
Toby 

On 17 September 2019 13:39:04 BST, Lars Ingebrigtsen <larsi <at> gnus.org> wrote:
>Alex Branham <alex.branham <at> gmail.com> writes:
>
>> -      (dotimes (i arg)
>> +      (dotimes (_i arg)
>
>Jonas said:
>
>> Toby, I am aware that I have suggested before that you use `_' as VAR
>in
>> calls to `dotimes' instead of `i'.  You rejected that suggestion,
>> claiming that the byte-compiler complaining about `i' not being used
>is
>> a bug.
>
>While my preference is to avoid compiler warnings (because having a
>warning-free build allows anybody working on the code to proceed with
>greater speed and confidence), I think that for ELPA code this should
>be
>up to the maintainer.  Since it seems like Toby is opposed to these
>changes, I'm closing this bug report.

-- 
Dr T. S. Cubitt
Reader (Associate Professor) in Quantum Information
Royal Society University Research Fellow
Department of Computer Science
University College London

email: tsc25 <at> cantab.net 
web:   www.dr-qubit.org




This bug report was last modified 5 years and 244 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.