Re: [PATCH 09/10] Make tdesc_type::u::u::fields an std::vector

Yao Qi
2017-11-02T10:02:16Z

Simon Marchi <simon.marchi@ericsson.com> writes:

> This patch makes the tdesc_type::u::u::fields an std::vector of
> tdesc_type_field. The difficulty here is that the vector is part of a
> union. Because of this, I made fields a pointer to a vector, and
> instantiate/destroy the vector if the type is one that uses this member
> of the union

Hi Simon,
Did you consider remove that union by sub-class tdesc_type. We can add
to new sub-classes, tdesc_type_with_fields and tdesc_type_vector, for
example?

--
Yao (齐尧)