Issue #3011 has been reported by dcb.

----------------------------------------
Bug #3011: dragonfly/sys/dev/netif/re/re.c: suspicious code ?
http://bugs.dragonflybsd.org/issues/3011

* Author: dcb
* Status: New
* Priority: Normal
* Assignee: 
* Category: 
* Target version: 
----------------------------------------


dragonfly/sys/dev/netif/re/re.c:6567] -> 
[dragonfly/sys/dev/netif/re/re.c:6567]: (style) Same expression on both sides 
of '||'.
[dragonfly/sys/dev/netif/re/re.c:21948] -> 
[dragonfly/sys/dev/netif/re/re.c:21948]: (style) Same expression on both sides 
of '||'.
[dragonfly/sys/dev/netif/re/re.c:21949] -> 
[dragonfly/sys/dev/netif/re/re.c:21949]: (style) Same expression on both sides 
of '||'.
[dragonfly/sys/dev/netif/re/re.c:22044] -> 
[dragonfly/sys/dev/netif/re/re.c:22044]: (style) Same expression on both sides 
of '||'.
[dragonfly/sys/dev/netif/re/re.c:22104] -> 
[dragonfly/sys/dev/netif/re/re.c:22104]: (style) Same expression on both sides 
of '||'.

Taking the first one as an example:

  if ((sc->re_type == MACFG_24) || (sc->re_type == MACFG_24) || (sc->re_type == 
MACFG_26))

MACFG_25 exists. Possible cut'n'paste error ?



-- 
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

Reply via email to