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: "gdb-patches at sourceware dot org" <gdb-patches at sourceware dot org>
- Date: Fri, 18 Oct 2013 18:18:03 +0100
- Subject: Re: [PATCH] Extra error message from update_watchpoint
- Authentication-results: sourceware.org; auth=none
- References: <5260FD66 dot 7090506 at broadcom dot com>
On 10/18/2013 10:20 AM, Andrew Burgess wrote:
> While working on this:
> URL
:-)
> @@ -1946,8 +1946,13 @@ update_watchpoint (struct watchpoint *b, int reparse)
> }
> }
> else if (!b->base.ops->works_in_software_mode (&b->base))
> - error (_("Expression cannot be implemented with "
> - "read/access watchpoint."));
> + {
> + if (!can_use_hw_watchpoints)
> + error (_("Software read/access watchpoints not supported."));
Hmm, is this really "not supported", or rather "explicitly disabled"?
I think I'd prefer:
error (_("Hardware watchpoints support disabled. "
"See set/show can-use-hw-watchpoints."));
which also hints towards the toggle. WDYT?
--
Pedro Alves