[Tarantool-patches] [PATCH v3] net.box: allow to store user-defined fields in future object

Igor Munkin imun at tarantool.org
Mon Aug 16 11:56:53 MSK 2021


Vova,

Thanks for the fixes!

On 16.08.21, Vladimir Davydov wrote:
> On Sat, Aug 14, 2021 at 02:17:00PM +0300, Igor Munkin wrote:
> > On 13.08.21, Vladimir Davydov wrote:

<snipped>

> > > +future.abc = nil
> > > +---
> > > +...
> > > +future.abc -- nil
> > > +---
> > > +- null
> > > +...
> > > +future.abc = nil
> > 
> > Looks like a typo: future.abc already equals to nil above.
> 
> It's actually not a typo. I wanted to ensure that deleting a key that
> has no value works fine. Added clarifying comments.

Now I get it, thanks! Feel free to proceed with the patch then.

> --

<snipped>

-- 
Best regards,
IM


More information about the Tarantool-patches mailing list