[PATCH v5] gdb: CTF support update
Weimin Pan
weimin.pan@oracle.com
Fri Dec 20 22:40:00 GMT 2019
On 12/19/2019 12:27 PM, Tom Tromey wrote:
>>>>>> Weimin Pan <weimin.pan@oracle.com> writes:
>> This patch addresses Tom Tromey's comments on the CTF reader:
>> * Use the type-safe registry for ctf_file_key;
>> * Drop "typedef" when defining "struct ctf_context";
>> * Use ANOFFSET with SECT_OFF_TEXT to get the text base address;
> Thanks for the patch.
>
>> +struct ctf_fp_info
>> + {
> This brace should be in the leftmost column.
>
>> + ctf_fp_info (ctf_file_t *cfp) : fp (cfp) {}
> Single-argument constructors should be marked "explicit".
>
> This patch is ok with these fixes. Thanks.
Moved the brace, added "explicit" to the constructor, and pushed. Thanks.
>
> Tom
More information about the Gdb-patches
mailing list