Simplify generic_val_print_func

This removes the call to val_print_scalar_formatted from
generic_val_print_func, allowing generic_value_print to call the
value-based variant instead.

gdb/ChangeLog
2020-03-13  Tom Tromey  <tom@tromey.com>

	* valprint.c (generic_val_print_func): Simplify.
	(generic_val_print, generic_value_print): Update.
This commit is contained in:
Tom Tromey
2020-03-13 17:39:52 -06:00
parent 65786af626
commit 4112d2e602
2 changed files with 26 additions and 20 deletions

View File

@ -1,3 +1,8 @@
2020-03-13 Tom Tromey <tom@tromey.com>
* valprint.c (generic_val_print_func): Simplify.
(generic_val_print, generic_value_print): Update.
2020-03-13 Tom Tromey <tom@tromey.com> 2020-03-13 Tom Tromey <tom@tromey.com>
* valprint.c (generic_val_print_flags): Remove. * valprint.c (generic_val_print_flags): Remove.

View File

@ -742,22 +742,16 @@ generic_val_print_func (struct type *type,
{ {
struct gdbarch *gdbarch = get_type_arch (type); struct gdbarch *gdbarch = get_type_arch (type);
if (options->format) gdb_assert (!options->format);
{
val_print_scalar_formatted (type, embedded_offset, /* FIXME, we should consider, at least for ANSI C language,
original_value, options, 0, stream); eliminating the distinction made between FUNCs and POINTERs to
} FUNCs. */
else fprintf_filtered (stream, "{");
{ type_print (type, "", stream, -1);
/* FIXME, we should consider, at least for ANSI C language, fprintf_filtered (stream, "} ");
eliminating the distinction made between FUNCs and POINTERs /* Try to print what function it points to, and its address. */
to FUNCs. */ print_address_demangle (options, gdbarch, address, stream, demangle);
fprintf_filtered (stream, "{");
type_print (type, "", stream, -1);
fprintf_filtered (stream, "} ");
/* Try to print what function it points to, and its address. */
print_address_demangle (options, gdbarch, address, stream, demangle);
}
} }
/* generic_val_print helper for TYPE_CODE_BOOL. */ /* generic_val_print helper for TYPE_CODE_BOOL. */
@ -971,8 +965,12 @@ generic_val_print (struct type *type,
case TYPE_CODE_FUNC: case TYPE_CODE_FUNC:
case TYPE_CODE_METHOD: case TYPE_CODE_METHOD:
generic_val_print_func (type, embedded_offset, address, stream, if (options->format)
original_value, options); val_print_scalar_formatted (type, embedded_offset,
original_value, options, 0, stream);
else
generic_val_print_func (type, embedded_offset, address, stream,
original_value, options);
break; break;
case TYPE_CODE_BOOL: case TYPE_CODE_BOOL:
@ -1084,8 +1082,11 @@ generic_value_print (struct value *val, struct ui_file *stream, int recurse,
case TYPE_CODE_FUNC: case TYPE_CODE_FUNC:
case TYPE_CODE_METHOD: case TYPE_CODE_METHOD:
generic_val_print_func (type, 0, value_address (val), stream, if (options->format)
val, options); value_print_scalar_formatted (val, options, 0, stream);
else
generic_val_print_func (type, 0, value_address (val), stream,
val, options);
break; break;
case TYPE_CODE_BOOL: case TYPE_CODE_BOOL: