Re: [PATCH 4/7] selftest/bpf: test_offload PEP8 format style fix

From: Jakub Kicinski
Date: Wed Dec 12 2018 - 13:41:08 EST

On Tue, 11 Dec 2018 20:56:04 +0900, Alice Ferrazzi wrote:
> Signed-off-by: Alice Ferrazzi <alice.ferrazzi@xxxxxxxxx>

Thanks, are you just running pylint to catch those? I tried in the
past but it's very noisy and it lacked "understanding" of some python3
modules. Would you mind giving us a quick 101 of how to catch such
errors? :)

> -def bpf_obj(name, sec=".text", path=bpf_test_dir,):
> +
> +def bpf_obj(name, sec=".text", path=bpf_test_dir, ):
> return "obj %s sec %s" % (os.path.join(path, name), sec)

I think this one is just a typo. bpf_obj() seems to always get called
with one param, no?