Re: [dpdk-dev] [PATCH] net/i40e: remove redundant statement and braces

2017-01-09 Thread Ferruh Yigit
On 1/9/2017 1:30 PM, Ferruh Yigit wrote: > On 1/9/2017 1:48 PM, Yong Wang wrote: >> In function "reassemble_packets()", the statement "end = secondlast;" >> is redundant since there is another assignment "start = end = NULL;" >> 3 lines below. BTW, I removed the redundant braces in the conditional

Re: [dpdk-dev] [PATCH] net/i40e: remove redundant statement and braces

2017-01-09 Thread Ferruh Yigit
On 1/9/2017 1:48 PM, Yong Wang wrote: > In function "reassemble_packets()", the statement "end = secondlast;" > is redundant since there is another assignment "start = end = NULL;" > 3 lines below. BTW, I removed the redundant braces in the conditional > statement "if (end->data_len > rxq->crc_len)

[dpdk-dev] [PATCH] net/i40e: remove redundant statement and braces

2017-01-09 Thread Yong Wang
In function "reassemble_packets()", the statement "end = secondlast;" is redundant since there is another assignment "start = end = NULL;" 3 lines below. BTW, I removed the redundant braces in the conditional statement "if (end->data_len > rxq->crc_len)". Signed-off-by: Yong Wang --- drivers/net