[DragonFlyBSD - Bug #2991] sys/dev/netif/age/if_age.c:628]: (style) Suspicious condition

bugtracker-admin at leaf.dragonflybsd.org bugtracker-admin at leaf.dragonflybsd.org
Wed Mar 29 00:37:49 PDT 2017


Issue #2991 has been updated by swildner.


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


----------------------------------------
Bug #2991: sys/dev/netif/age/if_age.c:628]: (style) Suspicious condition
http://bugs.dragonflybsd.org/issues/2991#change-13100

* Author: dcb
* Status: Closed
* Priority: Normal
* Assignee: 
* Category: 
* Target version: 
----------------------------------------
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;




-- 
You have received this notification because you have either subscribed to it, or are involved in it.
To change your notification preferences, please click here: http://bugs.dragonflybsd.org/my/account



More information about the Bugs mailing list