From: Daniel Jacobowitz <drow@false.org>
To: Pedro Alves <pedro@codesourcery.com>
Cc: gdb-patches@sourceware.org
Subject: Re: MI testsuite fix
Date: Mon, 14 Apr 2008 18:27:00 -0000 [thread overview]
Message-ID: <20080414182131.GC1968@caradoc.them.org> (raw)
In-Reply-To: <200804102102.07722.pedro@codesourcery.com>
On Thu, Apr 10, 2008 at 09:02:07PM +0100, Pedro Alves wrote:
> It is also arguable if that warning has any value, but in any case,
> we should be filtering warnings.
I am not sure this is true. Warning the user about this sort of
thing should imply something really wrong; otherwise, bothering
the user (who probably can not do anything about it except report it
to us) is not very helpful. And during the testsuite there shouldn't
be anything wrong that the user (i.e. the testsuite harness) can't
fix.
As for the warning itself, I'd approve a patch to remove it. I'd also
approve a patch to reuse some of the qOffsets machinery for this, but
that would take more testing... if you want to look at that,
keep in mind that SEC_LOAD doesn't mean the VMA is useful; you
also need to check !SEC_THREAD_LOCAL.
--
Daniel Jacobowitz
CodeSourcery
next prev parent reply other threads:[~2008-04-14 18:22 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-04-10 20:37 Pedro Alves
2008-04-10 21:42 ` Luis Machado
2008-04-10 22:10 ` Pedro Alves
2008-04-10 23:13 ` Pedro Alves
2008-04-14 18:27 ` Daniel Jacobowitz [this message]
2008-04-21 12:58 ` Pedro Alves
2008-04-21 14:04 ` Daniel Jacobowitz
2008-04-21 15:05 ` Pedro Alves
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=20080414182131.GC1968@caradoc.them.org \
--to=drow@false.org \
--cc=gdb-patches@sourceware.org \
--cc=pedro@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