[Tarantool-patches] [PATCH v2 05/10] box/error: don't set error created via box.error.new to diag
Alexander Turenko
alexander.turenko at tarantool.org
Thu Mar 26 21:07:26 MSK 2020
> > I know there was a discussion about construct vs create vs new.
>
> It was not really discussion, I accidentally didn't notice that
> box_error_construct() in fact allocated memory (meanwhile I thought
> it didn't). I've already renamed it to box_error_new().
>
> > Let me remind about approved name pairs:
> >
> > obj_create/obj_destroy - constructor/destructor, without
> > allocation/deallocation.
> >
> > obj_new/obj_delete - constructor/destructor with
> > allocation/deallocation
> >
> > init/free - library initialization/destruction
> >
> > _construct is not in the vocabulary. I'd avoid it.
>
> Ok, Vlad already guided me (but it would be nice to see this
> manual in our dev docs).
Should be there already: https://github.com/tarantool/doc/issues/1032
More information about the Tarantool-patches
mailing list