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