Re: [PATCH 02/43] Cleanup of the dwarf_expr_context constructor

Zoran Zaric via Gdb-patches
2021-04-28T10:19:14Z

>
> On 2021-03-01 9:45 a.m., Zoran Zaric via Gdb-patches wrote:
>> Move the initial values for dwarf_expr_context class data members
>> to the class declaration in expr.h.
>>
>> gdb/ChangeLog:
>>
>> * dwarf2/expr.c (dwarf_expr_context::dwarf_expr_context):
>> Remove initial data members values.
>> * dwarf2/expr.h (dwarf_expr_context): Add initial values
>> to the class data members.
>
> LGTM. I suggest you push this right away, it's a good cleanup and it
> will be one less patch to carry.
>
> Simon
>

Fair point.

Zoran