Nobody actually checked the return codes from Lst_Append and
authorMax Okumoto <okumoto@dragonflybsd.org>
Fri, 17 Dec 2004 08:07:49 +0000 (08:07 +0000)
committerMax Okumoto <okumoto@dragonflybsd.org>
Fri, 17 Dec 2004 08:07:49 +0000 (08:07 +0000)
Lst_Insert so don't return anything.

Taken-from: FreeBSD
Author: harti

usr.bin/make/lst.h
usr.bin/make/lst.lib/lstAppend.c
usr.bin/make/lst.lib/lstDupl.c
usr.bin/make/lst.lib/lstInsert.c

index e914eac..b8fec59 100644 (file)
@@ -38,7 +38,7 @@
  *
  *     from: @(#)lst.h 8.1 (Berkeley) 6/6/93
  * $FreeBSD: src/usr.bin/make/lst.h,v 1.9 1999/08/28 01:03:32 peter Exp $
- * $DragonFly: src/usr.bin/make/lst.h,v 1.16 2004/12/17 08:01:40 okumoto Exp $
+ * $DragonFly: src/usr.bin/make/lst.h,v 1.17 2004/12/17 08:07:49 okumoto Exp $
  */
 
 /*-
@@ -113,9 +113,9 @@ void                Lst_Destroy(Lst *, FreeProc *);
  * Functions to modify a list
  */
 /* Insert an element before another */
-ReturnStatus   Lst_Insert(Lst *, LstNode *, void *);
+void           Lst_Insert(Lst *, LstNode *, void *);
 /* Insert an element after another */
-ReturnStatus   Lst_Append(Lst *, LstNode *, void *);
+void           Lst_Append(Lst *, LstNode *, void *);
 /* Place an element at the front of a lst. */
 #define        Lst_AtFront(LST, D)     (Lst_Insert((LST), Lst_First(LST), (D)))
 /* Place an element at the end of a lst. */
index 8f1995b..77a5a82 100644 (file)
@@ -34,7 +34,7 @@
  * SUCH DAMAGE.
  *
  * $FreeBSD: src/usr.bin/make/lst.lib/lstAppend.c,v 1.6 1999/08/28 01:03:45 peter Exp $
- * $DragonFly: src/usr.bin/make/lst.lib/Attic/lstAppend.c,v 1.8 2004/12/17 00:02:57 okumoto Exp $
+ * $DragonFly: src/usr.bin/make/lst.lib/Attic/lstAppend.c,v 1.9 2004/12/17 08:07:49 okumoto Exp $
  *
  * @(#)lstAppend.c     8.1 (Berkeley) 6/6/93
  */
@@ -52,9 +52,6 @@
  * Lst_Append --
  *     Create a new node and add it to the given list after the given node.
  *
- * Results:
- *     SUCCESS if all went well.
- *
  * Arguments:
  *     l       affected list
  *     ln      node after which to append the datum
  *
  *-----------------------------------------------------------------------
  */
-ReturnStatus
+void
 Lst_Append(Lst *list, LstNode *ln, void *d)
 {
     LstNode *nLNode;
 
-    if (Lst_Valid(list) && (ln == NULL && Lst_IsEmpty(list))) {
-       goto ok;
-    }
-
-    if (!Lst_Valid(list) || Lst_IsEmpty(list)  || ! Lst_NodeValid(ln, list)) {
-       return (FAILURE);
-    }
-    ok:
-
     nLNode = emalloc(sizeof(*nLNode));
     nLNode->datum = d;
     nLNode->useCount = nLNode->flags = 0;
@@ -102,6 +90,4 @@ Lst_Append(Lst *list, LstNode *ln, void *d)
            list->lastPtr = nLNode;
        }
     }
-
-    return (SUCCESS);
 }
index a2ec1bf..67e7452 100644 (file)
@@ -34,7 +34,7 @@
  * SUCH DAMAGE.
  *
  * $FreeBSD: src/usr.bin/make/lst.lib/lstDupl.c,v 1.7 1999/08/28 01:03:49 peter Exp $
- * $DragonFly: src/usr.bin/make/lst.lib/Attic/lstDupl.c,v 1.10 2004/12/17 00:02:57 okumoto Exp $
+ * $DragonFly: src/usr.bin/make/lst.lib/Attic/lstDupl.c,v 1.11 2004/12/17 08:07:49 okumoto Exp $
  *
  * @(#)lstDupl.c       8.1 (Berkeley) 6/6/93
  */
@@ -82,14 +82,10 @@ Lst_Duplicate(Lst *list, DuplicateProc *copyProc)
 
     ln = list->firstPtr;
     while (ln != NULL) {
-       if (copyProc != NOCOPY) {
-           if (Lst_AtEnd(nl, (*copyProc)(ln->datum)) == FAILURE) {
-               return (NULL);
-           }
-       } else if (Lst_AtEnd (nl, ln->datum) == FAILURE) {
-           return (NULL);
-       }
-
+       if (copyProc != NOCOPY)
+           Lst_AtEnd(nl, (*copyProc)(ln->datum));
+       else
+           Lst_AtEnd(nl, ln->datum);
        ln = ln->nextPtr;
     }
 
index 6118983..ce6a1fd 100644 (file)
@@ -34,7 +34,7 @@
  * SUCH DAMAGE.
  *
  * $FreeBSD: src/usr.bin/make/lst.lib/lstInsert.c,v 1.6 1999/08/28 01:03:52 peter Exp $
- * $DragonFly: src/usr.bin/make/lst.lib/Attic/lstInsert.c,v 1.8 2004/12/17 00:02:57 okumoto Exp $
+ * $DragonFly: src/usr.bin/make/lst.lib/Attic/lstInsert.c,v 1.9 2004/12/17 08:07:49 okumoto Exp $
  *
  * @(#)lstInsert.c     8.1 (Berkeley) 6/6/93
  */
@@ -53,9 +53,7 @@
  *     Insert a new node with the given piece of data before the given
  *     node in the given list.
  *
- * Results:
- *     SUCCESS or FAILURE.
- *
+ * Parameters:
  *     l       list to manipulate
  *     ln      node before which to insert d
  *     d       datum to be inserted
  *
  *-----------------------------------------------------------------------
  */
-ReturnStatus
+void
 Lst_Insert(Lst *list, LstNode *ln, void *d)
 {
     LstNode *nLNode;   /* new lnode for d */
 
-    /*
-     * check validity of arguments
-     */
-    if (Lst_Valid(list) && (Lst_IsEmpty(list) && ln == NULL))
-       goto ok;
-
-    if (!Lst_Valid(list) || Lst_IsEmpty(list) || !Lst_NodeValid(ln, list)) {
-       return (FAILURE);
-    }
-
-    ok:
     nLNode = emalloc(sizeof(*nLNode));
-
     nLNode->datum = d;
     nLNode->useCount = nLNode->flags = 0;
 
@@ -103,6 +89,4 @@ Lst_Insert(Lst *list, LstNode *ln, void *d)
            list->firstPtr = nLNode;
        }
     }
-
-    return (SUCCESS);
 }