Re: [Tarantool-patches] [PATCH luajit v1 5/5] Use generic trace error for OOM during trace stitching.

[Sergey Kaplun via Tarantool-patches <tarantool-patches@dev.tarantool.org>]

Hi, Sergey!

On 13.06.24, Sergey Bronnikov wrote:
> Hi, Sergey
> 
> thanks for the answers.

<snipped>

> >>>    mockalloc.unmock()
> >> Same comment as in previous mail - let's avoid name 'mock' here.
> > But we actually mock the allocator here, don't we?
> > Thus, I renamed the new allocated function to avoid confusion.
> > If you insist, please suggest the correct name instead `mock` |
> > `unmock`.
> 
> We can replace "mock"/"unmock" to "{enable, disable} error injection".
> 
> Actually, I don't want to insist. I'm ok if it wouldn't confuse anyone.

Replaced in the 2nd verison of the patch-set [1].

> 
> >>>    
> > <snipped>
> >
> >>>    test:done(true)
> > [1]:https://lists.tarantool.org/pipermail/tarantool-patches/2024-June/029218.html
> >

[1]: https://lists.tarantool.org/pipermail/tarantool-patches/2024-June/029242.html

-- 
Best regards,
Sergey Kaplun