Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Daniel Jacobowitz <drow@false.org>
To: Eli Zaretskii <eliz@gnu.org>
Cc: Joel Brobecker <brobecker@adacore.com>,
	vladimir@codesourcery.com, 	chgenly@gmail.com,
	gdb-patches@sources.redhat.com
Subject: Re: gdb.texinfo patch for -var-list-children (2)
Date: Mon, 22 Jun 2009 15:02:00 -0000	[thread overview]
Message-ID: <20090622150201.GA10191@caradoc.them.org> (raw)
In-Reply-To: <E1MIbMV-0000AD-Bu@fencepost.gnu.org>

On Mon, Jun 22, 2009 at 12:42:31AM -0400, Eli Zaretskii wrote:
> So if from now on we are going to request only unified diffs, we
> should at least change this text.  FWIW, I don't think we should
> change this policy because it may prove inconvenient in some
> situations (e.g., the Patch utility produces reject files in context
> diff format).  But that's me.

I agree that we should accept context diffs.  I (strongly) prefer
unified, but there are some cases where a context diff is much easier
to read.

As for patch, I recently discovered --unified-reject-files - very useful.

-- 
Daniel Jacobowitz
CodeSourcery


  reply	other threads:[~2009-06-22 15:02 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-06-20  4:11 Chris Genly
2009-06-20  8:45 ` Vladimir Prus
2009-06-20 10:01   ` Eli Zaretskii
2009-06-22  3:18     ` Joel Brobecker
2009-06-22  4:42       ` Eli Zaretskii
2009-06-22 15:02         ` Daniel Jacobowitz [this message]
2009-06-23  1:00           ` Eli Zaretskii
2009-06-23  3:19             ` Daniel Jacobowitz
2009-06-20 15:18 ` Eli Zaretskii
2009-06-20 20:31   ` Chris Genly
2009-06-20 21:33     ` Chris Genly
2009-07-04 12:13       ` Eli Zaretskii
2009-06-20 23:12     ` Eli Zaretskii
2009-06-20 23:17     ` Samuel Bronson
2009-06-21 19:28       ` Chris Genly
2009-06-21 23:12         ` Eli Zaretskii

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=20090622150201.GA10191@caradoc.them.org \
    --to=drow@false.org \
    --cc=brobecker@adacore.com \
    --cc=chgenly@gmail.com \
    --cc=eliz@gnu.org \
    --cc=gdb-patches@sources.redhat.com \
    --cc=vladimir@codesourcery.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