[PATCH 08/66] Remove tui_list

Ruslan Kabatsayev b7.10110111@gmail.com
Mon Jun 24 15:12:00 GMT 2019


Hello,

On Mon, 24 Jun 2019 at 17:12, Pedro Alves <palves@redhat.com> wrote:
>
> On 6/23/19 11:42 PM, Tom Tromey wrote:
> > -static struct tui_list source_windows = {src_win_list, 0};
> > +static struct std::vector<tui_win_info *> source_windows;
>
> I usually prefer to drop "struct", but I can live with it.
>
> However, I have to say that "struct std::vector" looks weird.  :-)

BTW, if you compile a declaration like "struct
std::vector<tui_win_info*> source_windows;" with clang++ (with -Wall
option), you'll get a warning "struct 'vector' was previously declared
as a class [-Wmismatched-tags]", so it's better to either drop the
"struct" or change it to "class".

>
> (There are other instances in the patch.)
>
> Thanks,
> Pedro Alves

Regards,
Ruslan



More information about the Gdb-patches mailing list