This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [RFA, 2 of 3] save/restore process record, part 2 (core ops target)
- From: Eli Zaretskii <eliz at gnu dot org>
- To: Michael Snyder <msnyder at vmware dot com>
- Cc: gdb-patches at sourceware dot org, teawater at gmail dot com
- Date: Sat, 17 Oct 2009 10:05:31 +0200
- Subject: Re: [RFA, 2 of 3] save/restore process record, part 2 (core ops target)
- References: <4AD91CDB.5060002@vmware.com>
- Reply-to: Eli Zaretskii <eliz at gnu dot org>
> Date: Fri, 16 Oct 2009 18:24:43 -0700
> From: Michael Snyder <msnyder@vmware.com>
>
> + if (!tmp_to_resume)
> + error (_("Process record can't get to_resume."));
> + if (!tmp_to_wait)
> + error (_("Process record can't get to_wait."));
> + if (!tmp_to_store_registers)
> + error (_("Process record can't get to_store_registers."));
> + if (!tmp_to_insert_breakpoint)
> + error (_("Process record can't get to_insert_breakpoint."));
> + if (!tmp_to_remove_breakpoint)
> + error (_("Process record can't get to_remove_breakpoint."));
Can we rephrase these to be more user-friendly? As written, this text
is okay for debug printouts, but not for user-level error messages,
IMO. (Yes, I know this text was in the old version, but still...)
> + //if (strcmp (current_target.to_shortname, "record_core") == 0)
> + //record_load ();
C++-style comments?
Thanks.