This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [PATCH] Extra error message from update_watchpoint
- From: Pedro Alves <palves at redhat dot com>
- To: Andrew Burgess <aburgess at broadcom dot com>
- Cc: Eli Zaretskii <eliz at gnu dot org>, "gdb-patches at sourceware dot org" <gdb-patches at sourceware dot org>
- Date: Wed, 30 Oct 2013 18:48:36 +0000
- Subject: Re: [PATCH] Extra error message from update_watchpoint
- Authentication-results: sourceware.org; auth=none
- References: <5260FD66 dot 7090506 at broadcom dot com> <52616D4B dot 3020209 at redhat dot com> <526FE5E9 dot 3000909 at broadcom dot com> <83ob68dmfb dot fsf at gnu dot org> <526FF27B dot 8090209 at broadcom dot com> <5270F23E dot 7010602 at broadcom dot com> <83a9hqepgt dot fsf at gnu dot org> <52713E2F dot 2030709 at broadcom dot com>
On 10/30/2013 05:13 PM, Andrew Burgess wrote:
> "Can't set read/access watchpoint, software read/access watchpoints not
> supported, and hardware watchpoints disabled (see set/show
> can-use-hw-watchpoints)."
The mention of "software read/access watchpoints"
seems redundant to me. IMO, the shorter:
"Can't set read/access watchpoint with hardware watchpoint
support disabled (see set/show can-use-hw-watchpoints)."
has all the info necessary. WDYT?
If that's still too long, given that version says "disabled"
explicitly, then we might drop the "(see set/show
can-use-hw-watchpoints)" part. (I don't see an issue with
using more than one line though.)
--
Pedro Alves