From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from [87.239.111.99] (localhost [127.0.0.1]) by dev.tarantool.org (Postfix) with ESMTP id 8DE00562369; Thu, 3 Aug 2023 17:26:53 +0300 (MSK) DKIM-Filter: OpenDKIM Filter v2.11.0 dev.tarantool.org 8DE00562369 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=tarantool.org; s=dev; t=1691072813; bh=1laV0Rl6IUNXARnnoDtCYjaqb+1HpE3uUHhGqg4VrXc=; h=Date:To:Cc:References:In-Reply-To:Subject:List-Id: List-Unsubscribe:List-Archive:List-Post:List-Help:List-Subscribe: From:Reply-To:From; b=Mq5+i5aQbC66WaUlB1HQS8c0CgvvSg5B1MJ+wo73TSjNM8txGmtf7yfs/HrWW4l1D ka85uinr6I1FGkGPzFna/4pJ9hbnbwZdCoFxcRCX8H/rkMPddWuGov6XTRydl8Kupv LGKdA51MUWifdyE0sNgg7mKoZTt4qiDebcZhmSzo= Received: from smtpng1.i.mail.ru (smtpng1.i.mail.ru [94.100.181.251]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dev.tarantool.org (Postfix) with ESMTPS id 4962F575701 for ; Thu, 3 Aug 2023 17:26:26 +0300 (MSK) DKIM-Filter: OpenDKIM Filter v2.11.0 dev.tarantool.org 4962F575701 Received: by smtpng1.m.smailru.net with esmtpa (envelope-from ) id 1qRZHV-00069y-Ia; Thu, 03 Aug 2023 17:26:25 +0300 Message-ID: <5dd220fb-5712-2c8e-0501-f72560c12183@tarantool.org> Date: Thu, 3 Aug 2023 17:26:25 +0300 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.13.0 Content-Language: en-US To: Igor Munkin , Maxim Kokryashkin Cc: tarantool-patches@dev.tarantool.org References: <7e04eae9eadd01e9bad114c8f63557b1b33afee5.1691047132.git.imun@tarantool.org> In-Reply-To: <7e04eae9eadd01e9bad114c8f63557b1b33afee5.1691047132.git.imun@tarantool.org> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-Mailru-Src: smtp X-4EC0790: 10 X-7564579A: B8F34718100C35BD X-77F55803: 4F1203BC0FB41BD969E04B5EED670DC804E38A5F9341E5D89B81E0241E25E490182A05F53808504023AF688916A1F2956761D0377C4C4F7460B0373682B2DA3143B83244EEE9617F X-7FA49CB5: FF5795518A3D127A4AD6D5ED66289B5278DA827A17800CE7387B291F9AC4D188EA1F7E6F0F101C67BD4B6F7A4D31EC0BCC500DACC3FED6E28638F802B75D45FF8AA50765F790063775FF68B4B43662428638F802B75D45FF36EB9D2243A4F8B5A6FCA7DBDB1FC311F39EFFDF887939037866D6147AF826D8ED7A5A09765CDB916AD8C15E263B4BEB117882F4460429724CE54428C33FAD305F5C1EE8F4F765FCE2CCD8F0CAA010FB389733CBF5DBD5E9C8A9BA7A39EFB766F5D81C698A659EA7CC7F00164DA146DA9985D098DBDEAEC8643FE6A0CAC512C7F6B57BC7E6449061A352F6E88A58FB86F5D81C698A659EA7E827F84554CEF5019E625A9149C048EE33AC447995A7AD182BEBFE083D3B9BA73A03B725D353964BED96AA85C75E140D089D37D7C0E48F6C8AA50765F79006376A91CFDE938F542CEFF80C71ABB335746BA297DBC24807EABDAD6C7F3747799A X-C1DE0DAB: 0D63561A33F958A5A6D63FA01375C90D74A27E75A79504B75C98C8E9297B8EE0F87CCE6106E1FC07E67D4AC08A07B9B04AB4081B6A6C2E07CB5012B2E24CD356 X-C8649E89: 1C3962B70DF3F0ADE00A9FD3E00BEEDF3FED46C3ACD6F73ED3581295AF09D3DF87807E0823442EA2ED31085941D9CD0AF7F820E7B07EA4CF9207FF9E27949CB4A00FD17A7362A1282B55CDE99A15CC1D45AEBCFACA8A0F3B422E0C78D0E153558553022043D0BC9A8BCBCAE2017E32768534D49AD8764FFCE48CAC7CA610320002C26D483E81D6BE0DBAE6F56676BC7117BB6831D7356A2DEC5B5AD62611EEC62B5AFB4261A09AF0 X-D57D3AED: 3ZO7eAau8CL7WIMRKs4sN3D3tLDjz0dLbV79QFUyzQ2Ujvy7cMT6pYYqY16iZVKkSc3dCLJ7zSJH7+u4VD18S7Vl4ZUrpaVfd2+vE6kuoey4m4VkSEu530nj6fImhcD4MUrOEAnl0W826KZ9Q+tr5ycPtXkTV4k65bRjmOUUP8cvGozZ33TWg5HZplvhhXbhDGzqmQDTd6OAevLeAnq3Ra9uf7zvY2zzsIhlcp/Y7m53TZgf2aB4JOg4gkr2biojcir52QaMQ80eqCfBD/rKyQ== X-DA7885C5: 3A2229F28E10B27AE5BA414B00190E8204595CC1A7FE017148F98D5420010AAE262E2D401490A4A0DB037EFA58388B346E8BC1A9835FDE71 X-Mailru-Sender: 689FA8AB762F73930F533AC2B33E986BF6088B70FD5AB707D1498905526E8347DD788429FD8613638ED9BB8B05EE7B3AFB559BB5D741EB96D19CD4E7312BAA970A04DAD6CC59E3365FEEDEB644C299C0ED14614B50AE0675 X-Mras: Ok Subject: Re: [Tarantool-patches] [PATCH luajit 03/15] test: fix E201 and E202 errors by pycodestyle X-BeenThere: tarantool-patches@dev.tarantool.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Tarantool development patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , From: Sergey Bronnikov via Tarantool-patches Reply-To: Sergey Bronnikov Errors-To: tarantool-patches-bounces@dev.tarantool.org Sender: "Tarantool-patches" LGTM On 8/3/23 10:30, Igor Munkin wrote: > Fixed the only occurrence of E202 ("whitespace before ')'"), 7 > occurrences of E201 ("whitespace after '['") and 7 occurrences of E202 > ("whitespace before ']'") errors reported by pycodestyle[1][2]. > > [1]: https://www.flake8rules.com/rules/E201.html > [2]: https://www.flake8rules.com/rules/E202.html > > Signed-off-by: Igor Munkin > --- > src/luajit-gdb.py | 14 +++++++------- > src/luajit_lldb.py | 12 ++++++------ > 2 files changed, 13 insertions(+), 13 deletions(-) > > diff --git a/src/luajit-gdb.py b/src/luajit-gdb.py > index b84fdc93..198578b1 100644 > --- a/src/luajit-gdb.py > +++ b/src/luajit-gdb.py > @@ -195,7 +195,7 @@ 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 l in [L] + list(map(lambda l: lookup(l), ( > # LuaJIT main coro (see luajit/src/luajit.c) > 'globalL', > # Tarantool main coro (see tarantool/src/lua/init.h) > @@ -504,20 +504,20 @@ def dump_stack(L, base=None, top=None): > > def dump_gc(g): > gc = g['gc'] > - stats = [ '{key}: {value}'.format(key = f, value = gc[f]) for f in ( > + stats = ['{key}: {value}'.format(key = f, value = gc[f]) for f in ( > 'total', 'threshold', 'debt', 'estimate', 'stepmul', 'pause' > - ) ] > + )] > > - stats += [ 'sweepstr: {sweepstr}/{strmask}'.format( > + stats += ['sweepstr: {sweepstr}/{strmask}'.format( > sweepstr = gc['sweepstr'], > # String hash mask (size of hash table - 1). > strmask = g['strmask'] + 1, > - ) ] > + )] > > - stats += [ '{key}: {number} objects'.format( > + stats += ['{key}: {number} objects'.format( > key = stat, > number = handler(gc[stat]) > - ) for stat, handler in gclen.items() ] > + ) for stat, handler in gclen.items()] > > return '\n'.join(map(lambda s: '\t' + s, stats)) > > diff --git a/src/luajit_lldb.py b/src/luajit_lldb.py > index 3f636546..2887723b 100644 > --- a/src/luajit_lldb.py > +++ b/src/luajit_lldb.py > @@ -211,7 +211,7 @@ c_structs = { > } > > for cls in c_structs.keys(): > - globals()[cls] = type(cls, (Struct, ), {'metainfo': c_structs[cls]} ) > + globals()[cls] = type(cls, (Struct, ), {'metainfo': c_structs[cls]}) > > for cls in Struct.__subclasses__(): > ptr_name = cls.__name__ + 'Ptr' > @@ -394,16 +394,16 @@ def dump_gc(g): > 'total', 'threshold', 'debt', 'estimate', 'stepmul', 'pause' > )] > > - stats += [ 'sweepstr: {sweepstr}/{strmask}'.format( > + stats += ['sweepstr: {sweepstr}/{strmask}'.format( > sweepstr = gc.sweepstr, > # String hash mask (size of hash table - 1). > strmask = g.strmask + 1, > - ) ] > + )] > > - stats += [ '{key}: {number} objects'.format( > + stats += ['{key}: {number} objects'.format( > key = stat, > number = handler(getattr(gc, stat)) > - ) for stat, handler in gclen.items() ] > + ) for stat, handler in gclen.items()] > return '\n'.join(map(lambda s: '\t' + s, stats)) > > def mref(typename, obj): > @@ -424,7 +424,7 @@ 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_global(l), ( > + for l in [L] + list(map(lambda l: lookup_global(l), ( > # LuaJIT main coro (see luajit/src/luajit.c) > 'globalL', > # Tarantool main coro (see tarantool/src/lua/init.h)