Module Name:    src
Committed By:   joerg
Date:           Sat Sep 12 19:11:13 UTC 2015

Modified Files:
        src/usr.bin/config: pack.c

Log Message:
Negating an integer and comparing it to 1 is a fancy way of checking for
0, which in this case would be DEVI_ORPHAN. That clearly can't be the
intention here, so switch to using != as operation without negation
instead.


To generate a diff of this commit:
cvs rdiff -u -r1.9 -r1.10 src/usr.bin/config/pack.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.

Modified files:

Index: src/usr.bin/config/pack.c
diff -u src/usr.bin/config/pack.c:1.9 src/usr.bin/config/pack.c:1.10
--- src/usr.bin/config/pack.c:1.9	Wed Oct 29 17:14:50 2014
+++ src/usr.bin/config/pack.c	Sat Sep 12 19:11:13 2015
@@ -1,4 +1,4 @@
-/*	$NetBSD: pack.c,v 1.9 2014/10/29 17:14:50 christos Exp $	*/
+/*	$NetBSD: pack.c,v 1.10 2015/09/12 19:11:13 joerg Exp $	*/
 
 /*
  * Copyright (c) 1992, 1993
@@ -45,7 +45,7 @@
 #endif
 
 #include <sys/cdefs.h>
-__RCSID("$NetBSD: pack.c,v 1.9 2014/10/29 17:14:50 christos Exp $");
+__RCSID("$NetBSD: pack.c,v 1.10 2015/09/12 19:11:13 joerg Exp $");
 
 #include <sys/param.h>
 #include <stdlib.h>
@@ -119,7 +119,7 @@ pack(void)
 	 */
 	locspace = 0;
 	TAILQ_FOREACH(i, &alldevi, i_next) {
-		if (!i->i_active == DEVI_ACTIVE || i->i_collapsed)
+		if (i->i_active != DEVI_ACTIVE || i->i_collapsed)
 			continue;
 		if ((p = i->i_pspec) == NULL)
 			continue;

Reply via email to