This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [PATCH 44/55] Introduce cp_print_value_fields and c_value_print_struct
- From: Tom Tromey <tom at tromey dot com>
- To: Simon Marchi <simark at simark dot ca>
- Cc: Tom Tromey <tom at tromey dot com>, gdb-patches at sourceware dot org
- Date: Thu, 23 Jan 2020 16:47:47 -0700
- Subject: Re: [PATCH 44/55] Introduce cp_print_value_fields and c_value_print_struct
- References: <20191208182958.10181-1-tom@tromey.com> <20191208182958.10181-45-tom@tromey.com> <6fda3828-538f-af14-7a1e-ebcfaa27969e@simark.ca>
>>>>> "Simon" == Simon Marchi <simark@simark.ca> writes:
Simon> On 2019-12-08 1:29 p.m., Tom Tromey wrote:
>> @@ -385,6 +386,280 @@ cp_print_value_fields (struct type *type, struct type *real_type,
>> fprintf_filtered (stream, "}");
>> }
>>
>> +/* Mutually recursive subroutines of cp_print_value and c_val_print to
>> + print out a structure's fields: cp_print_value_fields and
>> + cp_print_value.
>> +
>> + TYPE, VALADDR, ADDRESS, STREAM, RECURSE, and OPTIONS have the same
>> + meanings as in cp_print_value and c_val_print.
>> +
>> + 2nd argument REAL_TYPE is used to carry over the type of the
>> + derived class across the recursion to base classes.
>> +
>> + DONT_PRINT is an array of baseclass types that we should not print,
>> + or zero if called from top level. */
Simon> This comment needs to be updated.
I did this.
Tom