From owner-p4-projects@FreeBSD.ORG Sat May 30 16:09:14 2009 Return-Path: Delivered-To: p4-projects@freebsd.org Received: by hub.freebsd.org (Postfix, from userid 32767) id 9387D10656B1; Sat, 30 May 2009 16:09:14 +0000 (UTC) Delivered-To: perforce@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 4F03E1065686 for ; Sat, 30 May 2009 16:09:14 +0000 (UTC) (envelope-from zec@fer.hr) Received: from repoman.freebsd.org (repoman.freebsd.org [IPv6:2001:4f8:fff6::29]) by mx1.freebsd.org (Postfix) with ESMTP id 3D5BE8FC1D for ; Sat, 30 May 2009 16:09:14 +0000 (UTC) (envelope-from zec@fer.hr) Received: from repoman.freebsd.org (localhost [127.0.0.1]) by repoman.freebsd.org (8.14.3/8.14.3) with ESMTP id n4UG9D3E067322 for ; Sat, 30 May 2009 16:09:13 GMT (envelope-from zec@fer.hr) Received: (from perforce@localhost) by repoman.freebsd.org (8.14.3/8.14.3/Submit) id n4UG9DqQ067320 for perforce@freebsd.org; Sat, 30 May 2009 16:09:13 GMT (envelope-from zec@fer.hr) Date: Sat, 30 May 2009 16:09:13 GMT Message-Id: <200905301609.n4UG9DqQ067320@repoman.freebsd.org> X-Authentication-Warning: repoman.freebsd.org: perforce set sender to zec@fer.hr using -f From: Marko Zec To: Perforce Change Reviews Cc: Subject: PERFORCE change 163092 for review X-BeenThere: p4-projects@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: p4 projects tree changes List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 30 May 2009 16:09:15 -0000 http://perforce.freebsd.org/chv.cgi?CH=163092 Change 163092 by zec@zec_tpx32 on 2009/05/30 16:09:10 Improve style. Affected files ... .. //depot/projects/vimage-commit2/src/sys/kern/kern_vimage.c#47 edit Differences ... ==== //depot/projects/vimage-commit2/src/sys/kern/kern_vimage.c#47 (text+ko) ==== @@ -268,7 +268,6 @@ return (ENOMEM); } } - return (error); } @@ -297,23 +296,24 @@ next_name++; if (namelen == 0) { if (strlen(next_name) == 0) - return(top); /* '.' == this vimage */ + return (top); /* '.' == this vimage */ else - return(NULL); + return (NULL); } } else namelen = strlen(name); if (namelen == 0) - return(NULL); - LIST_FOREACH(vip, &top->vi_child_head, vi_sibling) + return (NULL); + LIST_FOREACH(vip, &top->vi_child_head, vi_sibling) { if (strlen(vip->vi_name) == namelen && strncmp(name, vip->vi_name, namelen) == 0) { if (next_name != NULL) - return(vimage_by_name(vip, next_name)); + return (vimage_by_name(vip, next_name)); else - return(vip); + return (vip); } - return(NULL); + } + return (NULL); } static void @@ -355,21 +355,21 @@ /* Try to go deeper in the hierarchy */ next = LIST_FIRST(&where->vi_child_head); if (next != NULL) - return(next); + return (next); } do { /* Try to find next sibling */ next = LIST_NEXT(where, vi_sibling); if (!recurse || next != NULL) - return(next); + return (next); /* Nothing left on this level, go one level up */ where = where->vi_parent; } while (where != top->vi_parent); /* Nothing left to be visited, we are done */ - return(NULL); + return (NULL); } #endif /* VIMAGE */ /* User interface block */