Re: [PATCH 00/22] Convert ui-out subsystem to C++

Pedro Alves
2016-11-24T18:47:54Z

On 11/24/2016 04:07 PM, Simon Marchi wrote:
> On 2016-11-24 10:24, Simon Marchi wrote:
>> Hi all,
>>
>> This patchset converts the various data structures of the ui-out
>> susbsystem to
>> C++ classes. Except when stated otherwise, the goal is to preserve
>> the same
>> structure, which was already very class-like, and the same behaviour.
>> There
>> are also a few cleanup patches here and there.
>>
>> The patch "Class-ify ui_out" introduces many clang -Wmismatched-tags
>> warnings,
>> which John aimed to fix in another series. I am preparing a patch
>> that fixes
>> them, but since it will only be boring and mechanical changes, I
>> didn't want to
>> delay the submission of this series further.
>>
>> The series is also available here:
>>
>> https://github.com/simark/binutils-gdb/tree/cxx-ui-out-v1
>>
>> Simon
>
> My patch "[PATCH 21/22] Class-ify ui_out" bounced, being considered
> spam... is there anything I can do about that?
>
> Patches 19 and 20 are also missing, but I didn't receive a bounce notice
> about them, so maybe they'll come out of the void eventually.

BTW, all other patches (I haven't looked at the missing ones)
look good to me, apart from the nits I sent.

Thanks,
Pedro Alves