Hi,

This is the follow-on patch that I mentioned yesterday. Most of the 
changes are pretty mechanical. I would appreciate another pair of eagle 
eyes, though.

This patch "constifies" parse_exp_1 by making it argument const and 
strdup'ing it to rigidly enforce this. As Tom points out, this is 
necessary because nearly all of the language parsers *will* temporarily 
modify the input string. I have some patches for this that I will post 
next week.

I've tested this by rebuilding with --enable-targets=all on x86_64 
linux. There are no regressions against CVS HEAD on native, 
native-gdbserver, or cc-with-tweaks(-i).

This patch has the added side effect of constifying 
find_condition_and_thread, which is really what I've been after all 
along. :-)

Comments/questions/concerns?
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.