Re: [PATCH 1/2] Support the new PPC476 processor -- Arch Independent

Joel Brobecker
2009-12-30T05:11:57Z

> error (_("\
> Provided range can't be watched. Either the target watchpoint resources\n\
> don't support it, or hardware watchpoints are disabled in GDB\n\
> (see \"set can-use-hw-watchpoints\")."));

This is total nit-picking, but I suggest that we avoid the use of
contractions in error messages. I think this is too casual.

"can't" -> "cannot"
"don't" -> "do not"

> Here we are just imitating the behaviour (and error message) of the find
> command. Since the syntax is the same, we thought that it'd be confusing
> if GDB behaved differently in different commands with the same syntax.

I agree with that (consistency). I don't think I'd be fustrated at GDB
if I inserted the addresses in the wrong order and GDB errored out.

--
Joel