Re: [PATCH 21/22] Class-ify ui_out

Pedro Alves
2016-11-30T12:46:10Z

On 11/24/2016 07:18 PM, Simon Marchi wrote:

> -/* Create a ui_out object */
> +class ui_out_level
> +{
> + public:
>
> -extern ui_out *ui_out_new (ui_out_impl_base *impl, int flags = 0);
> + ui_out_level (ui_out_type type)
> + : m_field_count (0),
> + m_type (type)
> + {
> + }
> +
> + ui_out_type
> + type (void) const

Just a reminder: "explicit" and "(void)" in multiple places.

Otherwise LGTM.

Thanks,
Pedro Alves