Source-Changes-HG archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

[src/trunk]: src/sys/kern Goto still harmful, but use "goto alldone" in place...



details:   https://anonhg.NetBSD.org/src/rev/dad815c11fc8
branches:  trunk
changeset: 764368:dad815c11fc8
user:      dholland <dholland%NetBSD.org@localhost>
date:      Mon Apr 18 00:45:29 2011 +0000

description:
Goto still harmful, but use "goto alldone" in place of a loop break
for now anyway.

diffstat:

 sys/kern/vfs_lookup.c |  9 +++++----
 1 files changed, 5 insertions(+), 4 deletions(-)

diffs (40 lines):

diff -r 0144b30fc901 -r dad815c11fc8 sys/kern/vfs_lookup.c
--- a/sys/kern/vfs_lookup.c     Mon Apr 18 00:45:07 2011 +0000
+++ b/sys/kern/vfs_lookup.c     Mon Apr 18 00:45:29 2011 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: vfs_lookup.c,v 1.177 2011/04/18 00:45:07 dholland Exp $        */
+/*     $NetBSD: vfs_lookup.c,v 1.178 2011/04/18 00:45:29 dholland Exp $        */
 
 /*
  * Copyright (c) 1982, 1986, 1989, 1993
@@ -37,7 +37,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: vfs_lookup.c,v 1.177 2011/04/18 00:45:07 dholland Exp $");
+__KERNEL_RCSID(0, "$NetBSD: vfs_lookup.c,v 1.178 2011/04/18 00:45:29 dholland Exp $");
 
 #include "opt_magiclinks.h"
 
@@ -1184,11 +1184,11 @@
                        /*
                         * Success with no object returned means we're
                         * creating something and it isn't already
-                        * there. Break out of the main loop now so
+                        * there. Jump out of the main loop now so
                         * the code below doesn't have to test for
                         * foundobj == NULL.
                         */
-                       break;
+                       goto alldone;
                }
 
                /*
@@ -1351,6 +1351,7 @@
 
                break;
        }
+ alldone:
 
        /*
         * Done.



Home | Main Index | Thread Index | Old Index