Re: [PATCH next-next 0/2] net: vlan: two small refactors to make code more concise

From: Ziyang Xuan (William)
Date: Thu Sep 01 2022 - 21:12:39 EST




在 2022/9/1 21:28, Paolo Abeni 写道:
> Hello,
>
> On Wed, 2022-08-31 at 12:09 +0800, Ziyang Xuan wrote:
>> Give two small refactors to make code more concise.
>>
>> Ziyang Xuan (2):
>> net: vlan: remove unnecessary err variable in vlan_init_net()
>> net: vlan: reduce indentation level in __vlan_find_dev_deep_rcu()
>>
>> net/8021q/vlan.c | 5 +----
>> net/8021q/vlan_core.c | 22 +++++++++-------------
>> 2 files changed, 10 insertions(+), 17 deletions(-)
>
> The patches look correct to me, but I think is better to defer this
> kind of nun-functional refactors to some work actually doing new stuff,
> to avoid unneeded noise.
>
> Note that I merged a few other clean-up recently, but e.g. they at
> least formally removed some unneeded branch.
>
> Sorry, I'm not going to apply this series.

No problem, I will try to dig deeper.

>
> Cheers,
>
> Paolo
>
> .
>