From owner-p4-projects@FreeBSD.ORG Thu Jul 24 21:12:14 2008 Return-Path: Delivered-To: p4-projects@freebsd.org Received: by hub.freebsd.org (Postfix, from userid 32767) id 757F9106567D; Thu, 24 Jul 2008 21:12: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 38EAA1065678 for ; Thu, 24 Jul 2008 21:12:14 +0000 (UTC) (envelope-from trasz@freebsd.org) Received: from repoman.freebsd.org (repoman.freebsd.org [IPv6:2001:4f8:fff6::29]) by mx1.freebsd.org (Postfix) with ESMTP id 2F06A8FC1B for ; Thu, 24 Jul 2008 21:12:14 +0000 (UTC) (envelope-from trasz@freebsd.org) Received: from repoman.freebsd.org (localhost [127.0.0.1]) by repoman.freebsd.org (8.14.2/8.14.2) with ESMTP id m6OLCEwE032050 for ; Thu, 24 Jul 2008 21:12:14 GMT (envelope-from trasz@freebsd.org) Received: (from perforce@localhost) by repoman.freebsd.org (8.14.2/8.14.1/Submit) id m6OLCEWN032048 for perforce@freebsd.org; Thu, 24 Jul 2008 21:12:14 GMT (envelope-from trasz@freebsd.org) Date: Thu, 24 Jul 2008 21:12:14 GMT Message-Id: <200807242112.m6OLCEWN032048@repoman.freebsd.org> X-Authentication-Warning: repoman.freebsd.org: perforce set sender to trasz@freebsd.org using -f From: Edward Tomasz Napierala To: Perforce Change Reviews Cc: Subject: PERFORCE change 145848 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: Thu, 24 Jul 2008 21:12:14 -0000 http://perforce.freebsd.org/chv.cgi?CH=145848 Change 145848 by trasz@trasz_traszkan on 2008/07/24 21:11:35 Style change - replace ifs with switch. No functional change. Fever is not good for coding. ;-/ Affected files ... .. //depot/projects/soc2008/trasz_nfs4acl/TODO#14 edit .. //depot/projects/soc2008/trasz_nfs4acl/sys/kern/vfs_acl.c#5 edit Differences ... ==== //depot/projects/soc2008/trasz_nfs4acl/TODO#14 (text+ko) ==== @@ -48,8 +48,6 @@ - Make 'struct acl' variable size. -- Use switch() in vfs_acl.c:copyin_acl() instead of ifs. - - Check if Linux has acl_to_text_np; make sure we don't end up with something incompatible. (acl_to_any_text()?) ==== //depot/projects/soc2008/trasz_nfs4acl/sys/kern/vfs_acl.c#5 (text+ko) ==== @@ -129,17 +129,19 @@ int error; struct oldacl old; - /* Is it the new "struct acl"? */ - if (type != ACL_TYPE_ACCESS_OLD && type != ACL_TYPE_DEFAULT_OLD) - return (copyin(user_acl, kernel_acl, sizeof(struct acl))); + switch (type) { + case ACL_TYPE_ACCESS_OLD: + case ACL_TYPE_DEFAULT_OLD: + error = copyin(user_acl, &old, sizeof(struct oldacl)); + if (error) + break; + error = acl_copy_oldacl_into_acl(&old, kernel_acl); + break; - /* Nope, it's a "struct oldacl". */ - error = copyin(user_acl, &old, sizeof(struct oldacl)); - if (error) - return (error); + default: + error = copyin(user_acl, kernel_acl, sizeof(struct acl)); + } - error = acl_copy_oldacl_into_acl(&old, kernel_acl); - return (error); } @@ -149,14 +151,19 @@ int error; struct oldacl old; - if (type != ACL_TYPE_ACCESS_OLD && type != ACL_TYPE_DEFAULT_OLD) - return (copyout(kernel_acl, user_acl, sizeof(struct acl))); + switch (type) { + case ACL_TYPE_ACCESS_OLD: + case ACL_TYPE_DEFAULT_OLD: + error = acl_copy_acl_into_oldacl(kernel_acl, &old); + if (error) + break; - error = acl_copy_acl_into_oldacl(kernel_acl, &old); - if (error) - return (error); + error = copyout(&old, user_acl, sizeof(struct oldacl)); + break; - error = copyout(&old, user_acl, sizeof(struct oldacl)); + default: + error = copyout(kernel_acl, user_acl, sizeof(struct acl)); + } return (error); } @@ -164,13 +171,16 @@ static int type_unold(int type) { - if (type == ACL_TYPE_ACCESS_OLD) + switch (type) { + case ACL_TYPE_ACCESS_OLD: return (ACL_TYPE_ACCESS); - if (type == ACL_TYPE_DEFAULT_OLD) + case ACL_TYPE_DEFAULT_OLD: return (ACL_TYPE_DEFAULT); - return (type); + default: + return (type); + } } /*