On 15/09/2017 10:40, Alexey Kardashevskiy wrote:
> -static void flatview_init(FlatView *view)
> +static FlatView *flatview_alloc(void)
>  {
> +    FlatView *view;
> +
> +    view = g_new0(FlatView, 1);
>      view->ref = 1;
> -    view->ranges = NULL;
> -    view->nr = 0;
> -    view->nr_allocated = 0;
> +
> +    return view;
>  }

Maybe flatview_new for consistency with other functions?  Otherwise
definitely a good idea.

Paolo

Reply via email to