[dpdk-dev] [PATCH] fix checkpatch errors
Mcnamara, John
john.mcnamara at intel.com
Tue Jan 5 11:21:58 CET 2016
> -----Original Message-----
> From: Tan, Jianfeng
> Sent: Tuesday, January 5, 2016 2:21 AM
> To: Xie, Huawei; dev at dpdk.org
> Cc: Mcnamara, John; Stephen Hemminger; Yuanhan Liu
> Subject: RE: [PATCH] fix checkpatch errors
>
>
>
> > -----Original Message-----
> > From: Xie, Huawei
> > Sent: Monday, January 4, 2016 9:52 AM
> > To: dev at dpdk.org
> > Cc: Mcnamara, John; Tan, Jianfeng; Xie, Huawei
> > Subject: [PATCH] fix checkpatch errors
> >
> > Signed-off-by: Huawei Xie <huawei.xie at intel.com>
> ...
> > mbuf_poolname_build(sock_id, pool_name, sizeof(pool_name));
> > - return (rte_mempool_lookup((const char *)pool_name));
> > + return rte_mempool_lookup((const char *)pool_name);
>
> Hi Huawei,
>
> Assume this patch is to solve below error (reported by checkpatch):
> ERROR: return is not a function, parentheses are not required
>
> So maybe above fix is not necessary? Involve more people to discuss.
>
> And please include the error message in the commit message.
Hi Huawei,
The fix looks good and there was a similar patch applied previously for lib (from Ferruh):
6307b909b8e0 ("lib: remove extra parenthesis after return")
However, the commit message could be better. Maybe something like the above:
"remove extra parentheses".
John
--
More information about the dev
mailing list