X-Git-Url: http://git.tuebingen.mpg.de/?p=paraslash.git;a=blobdiff_plain;f=buffer_tree.c;h=56ab2b24cccd3e0b1f0bd786006cd3c08ab6424f;hp=b1eeb0363e3741d288656e8c5cea5aa9d186c0ab;hb=6fb1fc05a236d8732b0ae9581cc84d9958b3af76;hpb=41bf9b01ed2193a5fe368e40b8b1f16b5c4b5495

diff --git a/buffer_tree.c b/buffer_tree.c
index b1eeb036..56ab2b24 100644
--- a/buffer_tree.c
+++ b/buffer_tree.c
@@ -835,7 +835,7 @@ size_t btr_get_input_queue_size(struct btr_node *btrn)
 /**
  * Remove a node from the buffer tree, reconnecting parent and children.
  *
- * \param btrn The node to splice out.
+ * \param btrnp The node to splice out.
  *
  * This function is used by buffer tree nodes that do not exist during the
  * whole lifetime of the buffer tree. Unlike btr_remove_node(), calling
@@ -843,9 +843,9 @@ size_t btr_get_input_queue_size(struct btr_node *btrn)
  * but reconnects the buffer tree by making all child nodes of \a btrn children
  * of the parent of \a btrn.
  */
-void btr_splice_out_node(struct btr_node *btrn)
+void btr_splice_out_node(struct btr_node **btrnp)
 {
-	struct btr_node *ch, *tmp;
+	struct btr_node *btrn = *btrnp, *ch, *tmp;
 
 	assert(btrn);
 	PARA_NOTICE_LOG("splicing out %s\n", btrn->name);
@@ -862,7 +862,7 @@ void btr_splice_out_node(struct btr_node *btrn)
 			list_del(&ch->node);
 	}
 	assert(list_empty(&btrn->children));
-	btrn->parent = NULL;
+	*btrnp = NULL;
 }
 
 /**
@@ -1203,21 +1203,20 @@ int btr_node_status(struct btr_node *btrn, size_t min_iqs,
 	size_t iqs;
 
 	assert(btrn);
-	if (type != BTR_NT_LEAF) {
-		if (btr_no_children(btrn))
-			return -E_BTR_NO_CHILD;
-		if (btr_get_output_queue_size(btrn) > BTRN_MAX_PENDING)
-			return 0;
-	}
-	if (type != BTR_NT_ROOT) {
-		if (btr_eof(btrn))
-			return -E_BTR_EOF;
-		iqs = btr_get_input_queue_size(btrn);
-		if (iqs == 0) /* we have a parent, because not eof */
-			return 0;
-		if (iqs < min_iqs && !btr_no_parent(btrn))
-			return 0;
-	}
+	if (type != BTR_NT_LEAF && btr_no_children(btrn))
+		return -E_BTR_NO_CHILD;
+	if (type != BTR_NT_ROOT && btr_eof(btrn))
+		return -E_BTR_EOF;
+
+	if (btr_get_output_queue_size(btrn) > BTRN_MAX_PENDING)
+		return 0;
+	if (type == BTR_NT_ROOT)
+		return 1;
+	iqs = btr_get_input_queue_size(btrn);
+	if (iqs == 0) /* we have a parent, because not eof */
+		return 0;
+	if (iqs < min_iqs && !btr_no_parent(btrn))
+		return 0;
 	return 1;
 }