[dpdk-dev] [PATCH v2 3/5] net/hns3: adjust some code style

oulijun oulijun at huawei.com
Mon Nov 9 10:32:34 CET 2020



在 2020/11/7 0:44, Ferruh Yigit 写道:
> On 11/6/2020 3:51 AM, Lijun Ou wrote:
>> From: Hongbo Zheng <zhenghongbo3 at huawei.com>
>>
>> Here adjusts some code style for making the lines more
>> compact and removes some static check tool warnings.
>>
>> Signed-off-by: Hongbo Zheng <zhenghongbo3 at huawei.com>
>> Signed-off-by: Lijun Ou <oulijun at huawei.com>
>> ---
>> V1->V2:
>> - fix checkpatch warning
>> ---
>>   drivers/net/hns3/hns3_cmd.c    |   1 -
>>   drivers/net/hns3/hns3_ethdev.c |   6 +-
>>   drivers/net/hns3/hns3_ethdev.h |   1 -
>>   drivers/net/hns3/hns3_flow.c   |  37 +-
>>   drivers/net/hns3/hns3_intr.c   | 873 
>> +++++++++++++++++++++++++++--------------
>>   drivers/net/hns3/hns3_rxtx.c   |   1 -
>>   drivers/net/hns3/hns3_stats.c  |   1 -
>>   7 files changed, 597 insertions(+), 323 deletions(-)
> 
> This is a big(ish) patch with just whitespace changes, looks like 
> related to the personal preferences.
> This will create noise in the git history, and unnecessary conflict in 
> backporting patches.
> 
> I know it is tempting to fix here or there :), but lets fix those when 
> updating those lines for a functional change, and drop this patch.
> .
Requirements for some static check tools:
When struct and union members are initialized, each member is 
initialized on a separate line.
We think this requirement can be a better code style.
We can split the just whitespace changes with the above requirements?
because the Redundant blank lines  alarms.


> 


More information about the dev mailing list