From: Richard Stallman <rms@gnu.org>
To: Paul Eggert <eggert@cs.ucla.edu>
Cc: eliz@gnu.org, sb@dod.no, emacs-devel@gnu.org
Subject: Re: git history tracking across renames (and emacs support) (Was: The name gnus-cloud.el)
Date: Mon, 08 Jan 2018 21:55:11 -0500 [thread overview]
Message-ID: <E1eYk4F-000159-Ag@fencepost.gnu.org> (raw)
In-Reply-To: <745d2f02-8ca3-8962-0de4-053340c8fa63@cs.ucla.edu> (message from Paul Eggert on Mon, 8 Jan 2018 10:41:34 -0800)
[[[ To any NSA and FBI agents reading my email: please consider ]]]
[[[ whether defending the US Constitution against all enemies, ]]]
[[[ foreign or domestic, requires you to follow Snowden's example. ]]]
> If it's important enough to
> mention the change from extern to static in the commit message, the
> current guidelines would seem to imply that it's important enough to
> mention why the change was made in code comments.
Here's what the standard actually says:
If you think that a change
calls for explanation, you're probably right. Please do explain
it---but please put the full explanation in comments in the code,
where people will see it whenever they see the code.
So you're criticizing a practice which is NOT our standard.
Note also that it talks about "explanation". We recommend such
comments, NOT to state the facts of what changed, but to explain the
reasons for the change _when that seems useful to explain_.
Our advice does NOT suggest adding a comment to state the fact that
the variable was changed to static. However, if the reason for this
change was not evident, it would be good to explain that reason in
comments.
--
Dr Richard Stallman
President, Free Software Foundation (https://gnu.org, https://fsf.org)
Internet Hall-of-Famer (https://internethalloffame.org)
Skype: No way! See https://stallman.org/skype.html.
next prev parent reply other threads:[~2018-01-09 2:55 UTC|newest]
Thread overview: 219+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <m2po9o273c.fsf@linux-m68k.org>
[not found] ` <87374jd66s.fsf@lifelogs.com>
[not found] ` <87bmj6dda0.fsf@linux-m68k.org>
[not found] ` <87vahe911g.fsf@lifelogs.com>
[not found] ` <87374id7jy.fsf@linux-m68k.org>
[not found] ` <877ett8g7k.fsf@lifelogs.com>
[not found] ` <mvm8te9nuh4.fsf@suse.de>
[not found] ` <E1eOWht-0005nF-DD@fencepost.gnu.org>
[not found] ` <87a7yn7tqp.fsf@lifelogs.com>
[not found] ` <E1eOsjM-0000PB-02@fencepost.gnu.org>
[not found] ` <878te75xa1.fsf@lifelogs.com>
2017-12-15 21:26 ` The name gnus-cloud.el Richard Stallman
2017-12-16 22:34 ` Ted Zlatanov
2017-12-17 22:22 ` Richard Stallman
2017-12-17 22:31 ` Lars Ingebrigtsen
2017-12-17 22:57 ` Eric Abrahamsen
2017-12-18 21:15 ` Richard Stallman
2017-12-17 23:20 ` Stephen Berman
2017-12-18 21:17 ` Richard Stallman
2017-12-17 23:06 ` Stefan Monnier
2017-12-18 0:33 ` Ted Zlatanov
2017-12-18 3:31 ` Eli Zaretskii
2017-12-18 21:16 ` Richard Stallman
2017-12-19 3:37 ` Eli Zaretskii
2017-12-21 16:50 ` Richard Stallman
2017-12-21 17:13 ` Eli Zaretskii
2017-12-22 18:47 ` Richard Stallman
2017-12-22 18:57 ` Eli Zaretskii
2017-12-22 19:20 ` Eli Zaretskii
2017-12-23 14:58 ` Richard Stallman
2017-12-23 15:04 ` Eli Zaretskii
2017-12-24 20:34 ` Richard Stallman
2017-12-25 16:06 ` Eli Zaretskii
2017-12-25 22:27 ` Paul Eggert
2017-12-26 19:40 ` Richard Stallman
2018-07-10 13:16 ` Ted Zlatanov
2017-12-24 20:34 ` Richard Stallman
2017-12-25 16:09 ` Eli Zaretskii
2017-12-31 16:27 ` Steinar Bang
2017-12-31 16:54 ` Eli Zaretskii
2017-12-31 18:12 ` Óscar Fuentes
2017-12-31 18:26 ` Eli Zaretskii
2018-01-01 0:48 ` Steinar Bang
2018-01-01 4:40 ` Eli Zaretskii
2017-12-31 18:27 ` Andreas Schwab
2018-01-01 1:05 ` Steinar Bang
2018-01-01 4:42 ` Eli Zaretskii
2018-01-01 10:07 ` git history tracking across renames (and emacs support) (Was: The name gnus-cloud.el) Steinar Bang
2018-01-01 10:59 ` git history tracking across renames (and emacs support) Andreas Schwab
2018-01-01 21:40 ` Steinar Bang
2018-01-01 11:00 ` Andreas Schwab
2018-01-01 21:37 ` Steinar Bang
2018-01-01 11:06 ` Andreas Schwab
2018-01-01 21:36 ` Steinar Bang
2018-01-01 21:59 ` Andreas Schwab
2018-01-01 22:42 ` Steinar Bang
2018-01-01 22:04 ` Óscar Fuentes
2018-01-01 22:45 ` Noam Postavsky
2018-01-02 0:02 ` Óscar Fuentes
2018-01-02 1:15 ` git history tracking across renames (and emacs support) (Was: The name gnus-cloud.el) Richard Stallman
2018-01-02 8:06 ` Paul Eggert
2018-01-02 8:49 ` git history tracking across renames (and emacs support) Lars Ingebrigtsen
2018-07-10 13:14 ` Ted Zlatanov
2018-07-10 15:05 ` Marcin Borkowski
2018-07-10 15:36 ` Eli Zaretskii
2018-07-10 20:54 ` Ted Zlatanov
2018-07-10 23:04 ` Basil L. Contovounesios
2018-07-11 3:32 ` Eli Zaretskii
2018-07-11 3:53 ` Stefan Monnier
2018-07-11 4:44 ` Marcin Borkowski
2018-07-11 8:22 ` Eli Zaretskii
2018-07-11 13:54 ` git history tracking across renames (and emacs support), Re: git history tracking across renames (and emacs support), Re: git history tracking across renames (and emacs support), " Ted Zlatanov
2018-07-11 15:42 ` Eli Zaretskii
2018-07-11 16:09 ` Clément Pit-Claudel
2018-07-11 17:13 ` Eric Abrahamsen
2018-07-11 22:50 ` Richard Stallman
2018-07-11 23:04 ` Noam Postavsky
2018-07-12 23:35 ` Richard Stallman
2018-07-13 20:13 ` Jonas Bernoulli
2018-07-14 7:05 ` Eli Zaretskii
2018-07-11 17:31 ` Ted Zlatanov
2018-07-11 18:13 ` Eli Zaretskii
2018-07-11 20:14 ` Paul Eggert
2018-07-11 22:36 ` João Távora
2018-07-12 2:42 ` Eli Zaretskii
2018-07-12 9:04 ` João Távora
2018-07-12 13:48 ` Eli Zaretskii
2018-07-12 16:26 ` João Távora
2018-07-12 16:38 ` Eli Zaretskii
2018-07-12 16:47 ` João Távora
2018-07-12 16:59 ` Eli Zaretskii
2018-07-12 16:40 ` Stefan Monnier
2018-07-12 16:56 ` João Távora
2018-07-12 17:01 ` Eli Zaretskii
2018-07-12 19:37 ` Ted Zlatanov
2018-07-13 6:16 ` Eli Zaretskii
2018-07-12 17:17 ` Stefan Monnier
2018-07-12 23:12 ` João Távora
2018-07-13 18:47 ` Stefan Monnier
2018-07-14 17:36 ` [PATCH] New option to make C-x 4 a use file-less ChangeLog buffers (was Re: git history tracking across renames (and emacs support)) João Távora
2018-07-15 5:33 ` [PATCH] New option to make C-x 4 a use file-less ChangeLog buffers Basil L. Contovounesios
2018-07-15 10:41 ` João Távora
2018-07-15 14:11 ` Basil L. Contovounesios
2018-07-16 2:15 ` [PATCH] New option to make C-x 4 a use file-less ChangeLog buffers (was Re: git history tracking across renames (and emacs support)) Stefan Monnier
2018-07-16 10:10 ` João Távora
2018-07-16 12:33 ` Stefan Monnier
2018-07-16 14:02 ` João Távora
2018-07-16 15:44 ` Stefan Monnier
2018-07-16 18:00 ` João Távora
2018-07-16 18:22 ` Stefan Monnier
2018-07-16 18:50 ` João Távora
2018-07-16 21:09 ` Stefan Monnier
2018-07-21 10:45 ` Eli Zaretskii
2018-07-16 15:50 ` Eli Zaretskii
2018-07-16 23:02 ` João Távora
2018-07-21 10:42 ` Eli Zaretskii
2018-07-21 13:02 ` João Távora
2018-07-21 13:30 ` Eli Zaretskii
2018-07-21 15:08 ` João Távora
2018-07-21 16:13 ` Eli Zaretskii
2018-10-08 1:10 ` Ted Zlatanov
2018-10-08 1:31 ` João Távora
2018-07-12 13:36 ` git history tracking across renames (and emacs support) Stefan Monnier
2018-07-12 15:53 ` Basil L. Contovounesios
2018-07-12 16:00 ` Stefan Monnier
2018-07-12 16:48 ` Robert Pluim
2018-07-12 17:03 ` Stefan Monnier
2018-07-12 20:01 ` Robert Pluim
2018-07-12 20:06 ` Stefan Monnier
2018-07-12 20:28 ` Robert Pluim
2018-07-13 3:46 ` Stefan Monnier
2018-07-13 8:18 ` Robert Pluim
2018-07-13 12:16 ` Stefan Monnier
2018-07-13 13:07 ` Robert Pluim
2018-07-13 15:29 ` Clément Pit-Claudel
2018-07-13 15:49 ` Stefan Monnier
2018-07-12 16:40 ` João Távora
2018-07-12 16:59 ` Stefan Monnier
2018-07-12 23:33 ` João Távora
2018-07-13 18:55 ` Stefan Monnier
2018-07-13 19:45 ` João Távora
2018-07-13 20:33 ` Stefan Monnier
2018-07-11 17:08 ` Radon Rosborough
2018-07-11 17:56 ` Paul Eggert
2018-07-11 18:10 ` Eli Zaretskii
2018-07-11 22:51 ` Richard Stallman
2018-07-12 14:10 ` Eli Zaretskii
2018-07-11 22:51 ` Richard Stallman
2018-07-12 19:46 ` Ted Zlatanov
2018-07-12 23:03 ` Paul Eggert
2018-07-12 23:39 ` Richard Stallman
2018-07-11 18:25 ` Eli Zaretskii
2018-07-12 17:03 ` Radon Rosborough
2018-07-12 17:37 ` Eli Zaretskii
2018-07-13 4:33 ` Radon Rosborough
2018-07-13 8:14 ` Eli Zaretskii
2018-07-13 14:15 ` Paul Eggert
2018-07-13 14:39 ` Eli Zaretskii
2018-07-13 16:39 ` Stefan Monnier
2018-07-13 23:11 ` Richard Stallman
2018-07-14 6:58 ` Commit header lines (was: git history tracking across renames (and emacs support)) Eli Zaretskii
2018-07-14 16:09 ` Paul Eggert
2018-07-14 16:23 ` Eli Zaretskii
2018-07-14 21:57 ` Paul Eggert
2018-07-12 19:42 ` git history tracking across renames (and emacs support) Ted Zlatanov
2018-07-13 6:16 ` Eli Zaretskii
2018-01-02 16:15 ` git history tracking across renames (and emacs support) (Was: The name gnus-cloud.el) Eli Zaretskii
2018-01-03 2:50 ` Paul Eggert
2018-01-03 3:24 ` git history tracking across renames (and emacs support) Stefan Monnier
2018-01-03 15:36 ` Eli Zaretskii
2018-01-03 18:20 ` Stefan Monnier
2018-01-03 18:56 ` Eli Zaretskii
2018-01-03 20:17 ` Stefan Monnier
2018-01-03 20:43 ` Eli Zaretskii
2018-01-03 22:02 ` Stefan Monnier
2018-01-04 3:33 ` Eli Zaretskii
2018-01-04 5:00 ` Stefan Monnier
2018-01-04 6:39 ` Eli Zaretskii
2018-01-04 8:03 ` Paul Eggert
2018-01-04 14:25 ` Stefan Monnier
2018-01-04 16:13 ` Eli Zaretskii
2018-01-04 13:15 ` Dmitry Gutov
2018-01-05 3:51 ` Stefan Monnier
2018-01-03 18:29 ` Alan Mackenzie
2018-01-03 22:45 ` Stefan Monnier
2018-01-04 12:02 ` Alan Mackenzie
2018-01-04 18:05 ` Stefan Monnier
2018-01-03 15:32 ` git history tracking across renames (and emacs support) (Was: The name gnus-cloud.el) Eli Zaretskii
2018-01-03 20:11 ` git history tracking across renames (and emacs support) Stefan Monnier
2018-01-03 16:42 ` Stephen Leake
2018-01-04 11:46 ` Richard Stallman
2018-01-03 3:08 ` git history tracking across renames (and emacs support) (Was: The name gnus-cloud.el) Richard Stallman
2018-01-03 3:28 ` git history tracking across renames (and emacs support) Stefan Monnier
2018-01-08 2:16 ` git history tracking across renames (and emacs support) (Was: The name gnus-cloud.el) Richard Stallman
2018-01-08 4:22 ` Paul Eggert
2018-01-08 18:30 ` Eli Zaretskii
2018-01-08 18:41 ` Paul Eggert
2018-01-08 19:13 ` Eli Zaretskii
2018-01-08 19:29 ` Paul Eggert
2018-01-08 21:08 ` Eli Zaretskii
2018-01-09 2:55 ` Richard Stallman [this message]
2018-01-09 7:27 ` Paul Eggert
2018-01-10 3:03 ` Richard Stallman
2018-01-10 8:22 ` Paul Eggert
2018-01-10 15:32 ` Eli Zaretskii
2018-01-11 22:08 ` git history tracking across renames (and emacs support) Stefan Monnier
2018-01-12 8:21 ` Eli Zaretskii
2018-01-13 2:33 ` Paul Eggert
2018-01-13 5:13 ` Stefan Monnier
2018-01-13 8:31 ` Eli Zaretskii
2018-01-13 10:31 ` Tim Landscheidt
2018-01-13 17:42 ` Stefan Monnier
2018-01-13 19:22 ` Eli Zaretskii
2018-01-13 21:43 ` Stefan Monnier
2018-01-13 19:35 ` Paul Eggert
2018-01-13 19:42 ` Eli Zaretskii
2018-01-13 20:22 ` Paul Eggert
2018-01-14 3:31 ` Eli Zaretskii
2018-01-13 8:26 ` Eli Zaretskii
2018-01-16 2:16 ` git history tracking across renames (and emacs support) (Was: The name gnus-cloud.el) Richard Stallman
2018-01-09 2:54 ` Richard Stallman
2018-01-02 22:04 ` git history tracking across renames (and emacs support) Steinar Bang
2018-01-02 5:20 ` Stefan Monnier
2018-01-02 17:34 ` The name gnus-cloud.el Karl Fogel
2018-01-02 20:42 ` Chad Brown
2017-12-18 21:15 ` Richard Stallman
2017-12-19 6:09 ` John Wiegley
2017-12-19 7:42 ` Eli Zaretskii
2017-12-21 16:51 ` Richard Stallman
2017-12-19 9:09 ` joakim
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://www.gnu.org/software/emacs/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=E1eYk4F-000159-Ag@fencepost.gnu.org \
--to=rms@gnu.org \
--cc=eggert@cs.ucla.edu \
--cc=eliz@gnu.org \
--cc=emacs-devel@gnu.org \
--cc=sb@dod.no \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).