[PATCHES] unshadow python methods
Douglas Bagnall
douglas.bagnall at catalyst.net.nz
Thu Oct 25 20:14:21 UTC 2018
On 26/10/18 3:16 AM, Volker Lendecke wrote:
> On Thu, Oct 25, 2018 at 10:38:01PM +1300, Douglas Bagnall via samba-technical wrote:
>> In Python, if you go:
>>
>> def foo(x):
>> return x * 2
>>
>> def foo(x):
>> return x * 3
>>
>> the first definition will be silently ignored.
>>
>> We do this in a few places.
>>
>> CI: https://gitlab.com/samba-team/devel/samba/pipelines/34239680
>> (though if you look closely you will see I have split the first patch
>> there in two).
>
> Is there a lint-style checker for this? I'd imagine that this is
> pretty hard to spot in larger files.
>
These were found with pyflakes.
I am also looking at pylint and mypy, which complain more verbosely about
style but have no more semantic insight. Python linting is awful, all about
the placement of spaces when we want something like `gcc -Wall`.
Douglas
More information about the samba-technical
mailing list