public inbox for gcc@gcc.gnu.org
 help / color / mirror / Atom feed
From: kenner@vlsi1.ultra.nyu.edu (Richard Kenner)
To: njn@csse.unimelb.edu.au
Cc: andi@firstfloor.org, gcc@gcc.gnu.org
Subject: Re: Rant about ChangeLog entries and commit messages
Date: Tue, 04 Dec 2007 13:05:00 -0000	[thread overview]
Message-ID: <10712041305.AA18081@vlsi1.ultra.nyu.edu> (raw)
In-Reply-To: <Pine.GSO.4.61.0712041459400.22250@mulga.csse.unimelb.edu.au>

> I didn't say you cannot or should not use these tools.  But a good comment 
> on a piece of code sure beats a good commit message, which must be looked at 
> separately, and can be fragmented over multiple commits, etc.

I don't see one as "beating" the other because they have very different
purposes.  Sometimes you need one and sometimes you need the other.

The purpose of COMMENTS is to help somebody understand the code as it
stands at some point in time.  In most cases, that means saying WHAT the
code does and WHY (at some level) it does what it does.  Once in a while,
it also means saying why it DOESN'T do something, for example, if it might
appear that there's a simpler way of doing what the code is doing now but
it doesn't work for some subtle reason.  But it's NOT appropriate to put
into comments the historical remark that this code used to have a typo
which caused a miscompilation at some specific place.  However, the commit
log IS the place for that sort of note.

My view is that, in general, the comments are usually the most appropriate
place to put information about how the code currently works and the commit
log is generally the best place for information that contrasts how the code
currently works with how it used to work and provides the motivation for
making the change.  But there are exceptions to both of those generalizations.

  reply	other threads:[~2007-12-04 13:05 UTC|newest]

Thread overview: 84+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <2007-12-02-11-05-39+trackit+sam@rfc1149.net.suse.lists.egcs>
2007-12-02 18:33 ` Andi Kleen
     [not found] ` <jeodd9l7j1.fsf@sykes.suse.de.suse.lists.egcs>
     [not found]   ` <Pine.GSO.4.61.0712031739500.10932@mulga.csse.unimelb.edu.au.suse.lists.egcs>
2007-12-03 12:20     ` Andi Kleen
2007-12-04  4:03       ` Nicholas Nethercote
2007-12-04 13:05         ` Richard Kenner [this message]
2008-02-23  9:53           ` Rant about ChangeLog entries and commit messages - better to do something than just complain Tim Josling
2008-02-23 15:17             ` Laurent GUERBY
2008-02-23 15:54             ` Daniel Jacobowitz
2008-02-23 16:00               ` Andi Kleen
2008-02-28  5:16                 ` Alexandre Oliva
2007-12-04 10:19       ` Rant about ChangeLog entries and commit messages Robert Kiesling
     [not found] ` <200712022136.57819.ebotcazou@libertysurf.fr.suse.lists.egcs>
     [not found]   ` <4aca3dc20712021240k19f3eae5j66453276179c401a@mail.gmail.com.suse.lists.egcs>
     [not found]     ` <200712022355.23871.ebotcazou@libertysurf.fr.suse.lists.egcs>
     [not found]       ` <4aca3dc20712021621n39a036d2u21f471f231dfffe@mail.gmail.com.suse.lists.egcs>
     [not found]         ` <10712031329.AA20246@vlsi1.ultra.nyu.edu.suse.lists.egcs>
2007-12-03 16:34           ` Andi Kleen
2007-12-03 16:38             ` Richard Kenner
2007-12-02 10:05 Samuel Tardieu
2007-12-02 10:23 ` Andreas Schwab
2007-12-02 10:52   ` Samuel Tardieu
2007-12-02 12:04     ` Hans-Peter Nilsson
2007-12-02 23:28       ` Robert Dewar
2007-12-03  0:30         ` DJ Delorie
2007-12-03  4:06         ` Richard Kenner
2007-12-03  4:51           ` Daniel Berlin
2007-12-03 14:41             ` Richard Kenner
2007-12-02 11:43   ` Samuel Tardieu
2007-12-02 12:28   ` Richard Kenner
2007-12-02 12:43     ` Bernd Schmidt
2007-12-02 13:05       ` Eric Botcazou
2007-12-02 14:27         ` Robert Kiesling
2007-12-02 20:52           ` tim
2007-12-03  2:41             ` Robert Kiesling
2007-12-02 23:29         ` Robert Dewar
2007-12-02 20:28       ` Daniel Berlin
2007-12-02 20:36         ` Eric Botcazou
2007-12-02 20:40           ` Daniel Berlin
2007-12-02 22:55             ` Eric Botcazou
2007-12-03  0:21               ` Daniel Berlin
2007-12-03  6:16                 ` Eric Botcazou
2007-12-03 13:29                 ` Richard Kenner
2007-12-03 15:48                   ` Daniel Berlin
2007-12-03 16:20                     ` Richard Kenner
2007-12-03 16:50                     ` Robert Kiesling
2007-12-03 17:26                       ` Robert Dewar
2007-12-04 22:10                   ` Jeffrey Law
2007-12-16  3:03                   ` Alexandre Oliva
2007-12-16 10:25                     ` NightStrike
2007-12-16 13:04                       ` Alexandre Oliva
2007-12-16 18:52                         ` NightStrike
2007-12-25 19:35                     ` Tim Josling
2007-12-26  1:05                       ` Daniel Berlin
2007-12-26  2:17                       ` Richard Kenner
2007-12-26  2:31                         ` Robert Dewar
2007-12-26  4:53                           ` Richard Kenner
2007-12-26  6:00                       ` Alexandre Oliva
2007-12-02 20:59           ` tim
2007-12-02 23:32           ` Robert Dewar
2007-12-03  6:02             ` Eric Botcazou
2007-12-03  3:55           ` Richard Kenner
2007-12-02 23:31         ` Joseph S. Myers
2007-12-02 20:23     ` Daniel Berlin
2007-12-03  6:45   ` Nicholas Nethercote
2007-12-03  9:28     ` Andreas Schwab
2007-12-02 10:27 ` Eric Botcazou
2007-12-02 10:43   ` Samuel Tardieu
2007-12-02 10:50     ` Eric Botcazou
2007-12-02 11:14       ` Samuel Tardieu
2007-12-02 11:32         ` Eric Botcazou
2007-12-02 11:48           ` Samuel Tardieu
2007-12-02 12:25             ` Eric Botcazou
2007-12-02 23:03 ` Ben Elliston
2007-12-02 23:54   ` Daniel Jacobowitz
2007-12-03  4:03   ` Richard Kenner
2007-12-03 17:33 ` Diego Novillo
2007-12-03 17:37   ` Richard Kenner
2007-12-03 17:47   ` Bernd Schmidt
2007-12-03 17:58     ` Ian Lance Taylor
2007-12-03 18:20     ` Diego Novillo
2007-12-03 18:51       ` Richard Kenner
2007-12-03 18:58         ` Diego Novillo
2007-12-03 20:08           ` Tim Josling
2007-12-03 18:49     ` Richard Kenner
2007-12-04 16:45   ` Tom Tromey
2007-12-05 23:16     ` Ben Elliston
2007-12-05 23:35       ` Daniel Berlin
2007-12-06  0:29         ` Ben Elliston
2007-12-06  9:53           ` Andreas Schwab
2007-12-06  0:42       ` Robert Dewar

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

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=10712041305.AA18081@vlsi1.ultra.nyu.edu \
    --to=kenner@vlsi1.ultra.nyu.edu \
    --cc=andi@firstfloor.org \
    --cc=gcc@gcc.gnu.org \
    --cc=njn@csse.unimelb.edu.au \
    /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.
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).