This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [commit] Fix ARI violations in mips-linux-nat.c and mips-tdep.c
- From: "Maciej W. Rozycki" <macro at codesourcery dot com>
- To: Pedro Alves <palves at redhat dot com>
- Cc: Joel Brobecker <brobecker at adacore dot com>, <gdb-patches at sourceware dot org>
- Date: Tue, 22 May 2012 19:12:42 +0100
- Subject: Re: [commit] Fix ARI violations in mips-linux-nat.c and mips-tdep.c
- References: <20120302015615.GA30880@sourceware.org> <1330715778-30398-1-git-send-email-brobecker@adacore.com> <alpine.DEB.1.10.1203090032340.14492@tp.orcam.me.uk> <20120309041102.GA2853@adacore.com> <alpine.DEB.1.10.1203090724540.14492@tp.orcam.me.uk> <4F59E7B7.4060501@redhat.com> <alpine.DEB.1.10.1203091128080.14492@tp.orcam.me.uk>
On Fri, 9 Mar 2012, Maciej W. Rozycki wrote:
> > > What about the other issue I noted however, i.e.:
> > >
> > > gdb/event-loop.c: perror_with_name (("poll"));
> > >
> > > vs expected:
> > >
> > > gdb/event-loop.c: perror_with_name (_("poll"));
> > >
> > > (and so on) -- these clearly look like an oversight.
> >
> >
> > I don't think so. I think this is the way to silence the ARI for strings
> > that should not be translated. See:
>
> I didn't know that, thanks. In this case I think the same should be done
> with the "ptrace" reference in mips-linux-nat.c. I'll make a patch
> shortly.
This disappeared from my radar for a while, but came back now, and I have
decided to go for a more verbose error instead after all. I have checked
in this update now.
2012-05-22 Maciej W. Rozycki <macro@codesourcery.com>
gdb/
* mips-linux-nat.c (mips_linux_read_description): Use a more
verbose error message.
Maciej
gdb-mips-dsp-err.diff
Index: gdb-fsf-trunk-quilt/gdb/mips-linux-nat.c
===================================================================
--- gdb-fsf-trunk-quilt.orig/gdb/mips-linux-nat.c 2012-03-29 23:48:44.000000000 +0100
+++ gdb-fsf-trunk-quilt/gdb/mips-linux-nat.c 2012-05-20 14:09:08.325650594 +0100
@@ -447,7 +447,7 @@ mips_linux_read_description (struct targ
have_dsp = 0;
break;
default:
- perror_with_name (_("ptrace"));
+ perror_with_name (_("Couldn't check DSP support"));
break;
}
}