From: "André Pönitz" <apoenitz@trolltech.com>
To: gdb-patches@sources.redhat.com
Subject: Re: gdb sources
Date: Fri, 19 Sep 2008 15:46:00 -0000 [thread overview]
Message-ID: <200809191745.41960.apoenitz@trolltech.com> (raw)
In-Reply-To: <20080919152550.GA3593@caradoc.them.org>
On Friday 19 September 2008 17:25:50 Daniel Jacobowitz wrote:
> On Fri, Sep 19, 2008 at 05:20:13PM +0200, André Pönitz wrote:
> > > Please use diff -up for patches; with the default diff arguments, it's
> > > impossible to see what you've changed.
> >
> > I intented to do that but I somehow fumbled. Next try attached.
>
> Sorry, one more issue I didn't mention - please keep changelog entries
> as plain text, not a patch. They always conflict.
>
> > @@ -1,3 +1,18 @@
> > +
> > +2008-09-19 Andre Poenitz <apoenitz@trolltech.com>
> > +
> > + * symfile.c (allocate_symtab,allocate_psymtab): Accept a const argument.
> > + (allocate_psymtab): Likewise.
> > + (free_named_symtabs): Likewise.
> > + (deduce_language_from_filename): Likewise.
> > +
> > + * symfile.h (allocate_symtab): Update prototype.
> > + (allocate_psymtab): Likewise.
> > + (free_named_symtabs): Likewise.
> > + * symtab.h (deduce_language_from_filename): Likewise.
>
> It looks like you started rewriting this entry but didn't finish?
Not really. It's symfile.h in the beginning, and symtab.h in the end.
There's precedence for having no empty lines in the file contents
below, also for mixed (with and without empty lines) style within a
single Changelog entry, so I figured having no line might be
prefered if the stuff belongs close together (as in "Likewise,") and
an empty line belong where the differences are bigger (if they are
bigger in this case...)
Incidentally, sticking to rules would be much easier if there were
a blurb on the do's or don't's somewhere.
Andre'
next prev parent reply other threads:[~2008-09-19 15:46 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <200809121713.09226.apoenitz@trolltech.com>
[not found] ` <20080912153534.GA13672@caradoc.them.org>
2008-09-19 14:13 ` André Pönitz
2008-09-19 14:45 ` Daniel Jacobowitz
2008-09-19 15:21 ` André Pönitz
2008-09-19 15:26 ` Daniel Jacobowitz
2008-09-19 15:46 ` André Pönitz [this message]
2008-09-19 16:25 ` Daniel Jacobowitz
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=200809191745.41960.apoenitz@trolltech.com \
--to=apoenitz@trolltech.com \
--cc=gdb-patches@sources.redhat.com \
/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