[RFA] "constify" parse_exp_1
Keith Seitz
keiths@redhat.com
Fri Mar 8 01:02:00 GMT 2013
On 03/07/2013 04:22 PM, Pedro Alves wrote:
>
> I didn't get past the first hunk in the patch. :-)
Gah! That's what happens (sometimes) when you get too used to making
mechanical changes. My apologies!
> Your patch seems to make sym_name unused, even. There may be
> more instances of this in the patch. /me off to bed. :-)
I *think* that's the only one. The only other places where a
less-than-mechanical change is made is watch_command_1 (which is one
hairy beast), find_condition_and_thread (which is still pretty trivial),
and the tracepoint.c changes (which are almost as much a hairy beast).
So, allow me try again. Thank you! [And sleep well; this will be in your
INBOX in the morning. :-P]
Keith
ChangeLog
2013-03-07 Keith Seitz <keiths@redhat.com>
* ada-lang.c (ada_read_renaming_var_value): Pass const
pointer to expression string to parse_exp_1.
(craete_excep_cond_exprs): Likewise.
* ax-gdb.c (agent_eval_command_one): Likewise.
(maint_agent_printf_command): Likewise.
Constify much of the string handling/parsing.
* breakpoint.c (set_breakpoint_condition): Pass const
pointer to expression string to parse_exp_1.
(update_watchpoint): Likewise.
(parse_cmd_to_aexpr): Constify string handling.
Pass const pointer to parse_exp_1.
(init_breakpoint_sal): Pass const pointer to parse_exp_1.
(find_condition_and_thread): Likewise.
Make TOK const.
(watch_command_1): Constify string handling.
(update_breakpoint_location): Pass const pointer to
parse_exp_1.
* eval.c (parse_and_eval_address): Make EXP const.
(parse_to_comma_and_eval): Make EXPP const.
* expression.h (parse_expression): Make argument const.
(parse_exp_1): Make first argument const.
* findcmd.c (parse_find_args): Treat ARGS as const.
* linespec.c (parse_linespec): Pass const pointer to
linespec_expression_to_pc.
(linespec_expression_to_pc): Make EXP_PTR const.
* parse.c (parse_exp_1): Make STRINGPTR const.
Make a copy of the expression to pass to parse_exp_in_context until
this whole interface can be constified.
(parse_expression): Make STRING const.
* printcmd.c (ui_printf): Treat ARG as const.
Handle const strings.
* tracepoint.c (validate_actionline): Pass const pointer to
all calls to parse_exp_1.
(encode_actions_1): Likewise.
* value.h (parse_to_comma_and_eval): Make argument const.
(parse_and_eval_address): Likewise.
* varobj.c (varobj_create): Pass const pointer to parse_exp_1.
(varobj_set_value): Likewise.
* cli/cli-cmds.c (disassemble_command): Treat ARG as const and
constify string handling.
Pass const pointers to parse_and_eval_address and
parse_to_comman_and_eval.
* cli/cli-utils.c (skip_to_space): Rename to ...
(skip_to_space_const): ... this. Handle const strings.
* cli/cli-utils.h (skip_to_space): Turn into macro which invokes
skip_to_space_const.
(skip_to_space_const): Declare.
* common/format.c (parse_format_string): Make ARG const.
Handle const strings.
* common/format.h (parse_format_string): Make ARG const.
* gdbserver/ax.c (ax_printf): Make FORMAT const.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: constify-parse_exp_1.patch
Type: text/x-patch
Size: 27008 bytes
Desc: not available
URL: <http://sourceware.org/pipermail/gdb-patches/attachments/20130308/46ba9e48/attachment.bin>
More information about the Gdb-patches
mailing list