linker_file_add_dependancy() never fails so change its return type to void and
authorNicolas Thery <nth@dragonflybsd.org>
Wed, 6 Feb 2008 22:37:46 +0000 (22:37 +0000)
committerNicolas Thery <nth@dragonflybsd.org>
Wed, 6 Feb 2008 22:37:46 +0000 (22:37 +0000)
simplify calling sequences.

sys/kern/kern_linker.c
sys/kern/link_aout.c
sys/kern/link_elf.c
sys/sys/linker.h

index 9bd323d..4791fd2 100644 (file)
@@ -24,7 +24,7 @@
  * SUCH DAMAGE.
  *
  * $FreeBSD: src/sys/kern/kern_linker.c,v 1.41.2.3 2001/11/21 17:50:35 luigi Exp $
- * $DragonFly: src/sys/kern/kern_linker.c,v 1.42 2008/01/10 10:31:48 matthias Exp $
+ * $DragonFly: src/sys/kern/kern_linker.c,v 1.43 2008/02/06 22:37:46 nth Exp $
  */
 
 #include "opt_ddb.h"
@@ -483,7 +483,7 @@ out:
     return error;
 }
 
-int
+void
 linker_file_add_dependancy(linker_file_t file, linker_file_t dep)
 {
     linker_file_t* newdeps;
@@ -498,8 +498,6 @@ linker_file_add_dependancy(linker_file_t file, linker_file_t dep)
     file->deps = newdeps;
     file->deps[file->ndeps] = dep;
     file->ndeps++;
-
-    return 0;
 }
 
 /*
index acbe7cc..a8a7c60 100644 (file)
@@ -24,7 +24,7 @@
  * SUCH DAMAGE.
  *
  * $FreeBSD: src/sys/kern/link_aout.c,v 1.26 1999/12/24 15:33:36 bde Exp $
- * $DragonFly: src/sys/kern/link_aout.c,v 1.23 2008/01/05 14:02:38 swildner Exp $
+ * $DragonFly: src/sys/kern/link_aout.c,v 1.24 2008/02/06 22:37:46 nth Exp $
  */
 
 #define FREEBSD_AOUT   1
@@ -348,9 +348,7 @@ load_dependancies(linker_file_t lf)
        error = linker_load_file(name, &lfdep);
        if (error)
            goto out;
-       error = linker_file_add_dependancy(lf, lfdep);
-       if (error)
-           goto out;
+       linker_file_add_dependancy(lf, lfdep);
        off = sodp->sod_next;
     }
 
index 142239c..3775b6a 100644 (file)
@@ -24,7 +24,7 @@
  * SUCH DAMAGE.
  *
  * $FreeBSD: src/sys/kern/link_elf.c,v 1.24 1999/12/24 15:33:36 bde Exp $
- * $DragonFly: src/sys/kern/link_elf.c,v 1.27 2008/01/05 16:56:17 nth Exp $
+ * $DragonFly: src/sys/kern/link_elf.c,v 1.28 2008/02/06 22:37:46 nth Exp $
  */
 
 #include <sys/param.h>
@@ -748,9 +748,7 @@ load_dependancies(linker_file_t lf)
            error = linker_load_file(name, &lfdep);
            if (error)
                goto out;
-           error = linker_file_add_dependancy(lf, lfdep);
-           if (error)
-               goto out;
+           linker_file_add_dependancy(lf, lfdep);
        }
     }
 
index 2f57a1b..a7f8c78 100644 (file)
@@ -24,7 +24,7 @@
  * SUCH DAMAGE.
  *
  * $FreeBSD: src/sys/sys/linker.h,v 1.17.2.1 2002/03/11 01:13:53 dd Exp $
- * $DragonFly: src/sys/sys/linker.h,v 1.10 2006/12/23 00:27:03 swildner Exp $
+ * $DragonFly: src/sys/sys/linker.h,v 1.11 2008/02/06 22:37:46 nth Exp $
  */
 
 #ifndef _SYS_LINKER_H_
@@ -191,7 +191,7 @@ int linker_file_unload(linker_file_t _file);
 /*
  * Add a dependancy to a file.
  */
-int linker_file_add_dependancy(linker_file_t _file, linker_file_t _dep);
+void linker_file_add_dependancy(linker_file_t _file, linker_file_t _dep);
 
 /*
  * Lookup a symbol in a file.  If deps is TRUE, look in dependancies