Project

General

Profile

Actions

Bug #2992

closed

sys/netgraph7/netgraph/ng_base.c:590]: (style) Suspicious condition

Added by dcb about 7 years ago. Updated about 7 years ago.

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

0%

Estimated time:

Description

sys/netgraph7/netgraph/ng_base.c:590]: (style) Suspicious condition (assignment + comparison); Clarify expression with parentheses.

Source code is

if ((error = ((*type->constructor)(*nodepp)) != 0)) {

maybe better code

if ((error = ((*type->constructor)(*nodepp))) != 0) {
Actions #1

Updated by dillon about 7 years ago

  • Status changed from New to Closed

fix committed to master

Actions

Also available in: Atom PDF