[Tarantool-patches] [PATCH luajit 14/15] test: fix E741 errors by pycodestyle
Sergey Bronnikov
sergeyb at tarantool.org
Mon Aug 7 16:45:52 MSK 2023
Igor,
On 8/7/23 14:00, Igor Munkin wrote:
<snipped>
>>> diff --git a/src/luajit-gdb.py b/src/luajit-gdb.py
>>> index a0c8f24e..2e0b145b 100644
>>> --- a/src/luajit-gdb.py
>>> +++ b/src/luajit-gdb.py
>>> @@ -233,15 +233,15 @@ def L(L=None):
>>> # lookup a symbol for the main coroutine considering the host app
>>> # XXX Fragile: though the loop initialization looks like a crap but it
>>> # respects both Python 2 and Python 3.
>>> - for l in [L] + list(map(lambda l: lookup(l), (
>>> + for coroutine in [L] + list(map(lambda main: lookup(main), (
>> it is not obvious why variable "l" was replaced by "coroutine".
>>
>> Could you elaborate? The same below
> It's quite clear (at least for me): lua_State represents Lua coroutine
> within low level space. However, I've already fixed it to "lstate" as
> Max suggested (you can find the incremental diff in my reply).
>
>
Thanks for explanation! LGTM now
More information about the Tarantool-patches
mailing list