This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [patch 1/2] Convert hardware watchpoints to use breakpoint_ops
- From: Joel Brobecker <brobecker at adacore dot com>
- To: Thiago Jung Bauermann <bauerman at br dot ibm dot com>
- Cc: gdb-patches at sourceware dot org
- Date: Thu, 18 Nov 2010 09:18:34 -0800
- Subject: Re: [patch 1/2] Convert hardware watchpoints to use breakpoint_ops
- References: <1282074071.2606.702.camel@hactar> <201010161843.43062.pedro@codesourcery.com> <1287534691.2686.17.camel@hactar> <20101115222310.GB4434@adacore.com> <1289933508.3202.13.camel@hactar>
> I kept the wording as similar as possible to the original. Your
> suggestion is then to have the code below instead?
>
> + if (val == 1)
> + warning (_("\
> +Error inserting catchpoint %d: Your system does not support this type of catchpoint."),
> + bpt->owner->number);
> + else
> + warning (_("Error inserting catchpoint %d."), bpt->owner->number);
Yep!
> What about these additional comments (I'll send the updated patch after
> understanding your suggestion above)?
They look good to me. IMO, you can even drop the first hunk,
documenting the return value inside the target_ops vector definition.
I'd rather have no documentation than an incomplete duplicate of
the actual documentation located elsewhere. The idea is that, if
there is no documentation, then it might help trigger a search of
that documentation elsewhere. If there is some, then we might not
have the idea to look elsewhere for the rest...
--
Joel