Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
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'




  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