Re: [PATCH] Extra error message from update_watchpoint

Pedro Alves
2013-10-30T18:48:43Z

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