Project

General

Profile

Bug #2991

sys/dev/netif/age/if_age.c:628]: (style) Suspicious condition

Added by dcb over 2 years ago. Updated over 2 years ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Category:
-
Target version:
-
Start date:
03/28/2017
Due date:
% Done:

0%

Estimated time:

Description

sys/dev/netif/age/if_age.c:628]: (style) Suspicious condition (assignment + comparison); Clarify expression with parentheses.

Source code is

if ((error = age_dma_alloc(sc) != 0))
goto fail;

Maybe better code

if ((error = age_dma_alloc(sc)) != 0)
goto fail;

History

#1

Updated by dcb over 2 years ago

Same thing in different file:

dragonfly/sys/dev/netif/ale/if_ale.c:599]: (style) Suspicious condition (assignment + comparison); Clarify expression with parentheses.

#2

Updated by dillon over 2 years ago

  • Status changed from New to Closed

fix committed to master

#3

Updated by swildner over 2 years ago

I've pushed the ale(4) change too.

Also available in: Atom PDF