From: Gabriel Dos Reis <gdr@cs.tamu.edu>
To: Daniel Jacobowitz <dan@codesourcery.com>
Cc: gdb-patches@sourceware.org
Subject: Re: PATCH to gdb/linux-nat.c
Date: Fri, 09 Feb 2007 19:39:00 -0000 [thread overview]
Message-ID: <87irebm86x.fsf@soliton.cs.tamu.edu> (raw)
In-Reply-To: <20070209192353.GA30049@nevyn.them.org>
Daniel Jacobowitz <dan@codesourcery.com> writes:
| On Fri, Feb 09, 2007 at 10:37:49AM -0600, Gabriel Dos Reis wrote:
| > Daniel Jacobowitz <dan@codesourcery.com> writes:
| >
| > | On Fri, Feb 09, 2007 at 03:08:38AM -0600, Gabriel Dos Reis wrote:
| > | > 2007-02-09 Gabriel Dos Reis <gdr@integrable-solutions.net>
| > | >
| > | > * linux-nat.c (linux_nat_find_memory_regions): Don't check the
| > | > address of 'filename'; it is always non null.
| > |
| > | OK, thanks. Do you have write access to GDB? If not, do you want it?
| >
| > A while ago, when libstdc++-v3 was a standalone project, I had an
| > account on sourceware. I think things have changed over the years, so
| > yes, I'll welcome a write access t GDB. Thanks!
|
| OK, now you have write access. Please commit this, and post and commit
| a patch adding yourself to the write-after-approval list in
| gdb/MAINTAINERS. Thanks for the patch!
Great; that was quick! Many thanks.
One more favor: Would you mind looking at my other patch? The GCC PR
I filled was demonstrated to be a duplicate and nothing happened
since the first PR was filled; I believe GDB+GCC-4.2.0 is affected too. I
think the patch I sent works around the problem within GDB and
"improves" the const-correctness of the two functions involved.
Thanks,
-- Gaby
next prev parent reply other threads:[~2007-02-09 19:39 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-02-09 9:08 Gabriel Dos Reis
2007-02-09 13:49 ` Daniel Jacobowitz
2007-02-09 16:38 ` Gabriel Dos Reis
2007-02-09 19:24 ` Daniel Jacobowitz
2007-02-09 19:39 ` Gabriel Dos Reis [this message]
2007-02-09 19:41 ` Daniel Jacobowitz
2007-02-09 19:44 ` Gabriel Dos Reis
2007-02-09 20:00 ` Joel Brobecker
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=87irebm86x.fsf@soliton.cs.tamu.edu \
--to=gdr@cs.tamu.edu \
--cc=dan@codesourcery.com \
--cc=gdb-patches@sourceware.org \
/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