compilation fixes for gcc 4.0

This one causes most doubts in me, as it is a function declared inside body of another function.

Is there really a need to have such mad code in reiser4 fs ?? This issue was raised already on reiser4 mailing list. Here it's only for reviev and opinions, if no doubts, please apply.

@@ -1088,6 +1088,20 @@

}
#endif

+/* true if @key is left delimiting key of @node */ +static int key_is_ld(znode * node, const reiser4_key * key) {
+ int ld;
+
+ assert("nikita-1716", node != NULL);
+ assert("nikita-1758", key != NULL);
+

+        RLOCK_DK(znode_get_tree(node));
+        assert("nikita-1759", znode_contains_key(node, key));
+        ld = keyeq(znode_get_ld_key(node), key);
+        RUNLOCK_DK(znode_get_tree(node));
+        return ld;

+}
+
/* Process one node during tree traversal.

This is called by cbk_level_lookup(). */
@@ -1107,19 +1121,6 @@

       /* result */
       int result;

- /* true if @key is left delimiting key of @node */
- static int key_is_ld(znode * node, const reiser4_key * key) {
- int ld;
-
- assert("nikita-1716", node != NULL);
- assert("nikita-1758", key != NULL);
-

-                RLOCK_DK(znode_get_tree(node));
-                assert("nikita-1759", znode_contains_key(node, key));
-                ld = keyeq(znode_get_ld_key(node), key);
-                RUNLOCK_DK(znode_get_tree(node));
-                return ld;
-       }
       assert("nikita-379", h != NULL);

       active = h->active_lh->node;

--
GJ

I need to change email client :-)
Patch attached.

--
GJ

@@ -1088,6 +1088,20 @@
 }
 #endif
 
+/* true if @key is left delimiting key of @node */
+static int key_is_ld(znode * node, const reiser4_key * key) {
+	int ld;
+
+	 assert("nikita-1716", node != NULL);
+	 assert("nikita-1758", key != NULL);
+
+	 RLOCK_DK(znode_get_tree(node));
+	 assert("nikita-1759", znode_contains_key(node, key));
+	 ld = keyeq(znode_get_ld_key(node), key);
+	 RUNLOCK_DK(znode_get_tree(node));
+	 return ld;
+}
+
 /* Process one node during tree traversal.
 
    This is called by cbk_level_lookup(). */

@@ -1107,19 +1121,6 @@
 	/* result */
 	int result;
 
-	/* true if @key is left delimiting key of @node */
-	static int key_is_ld(znode * node, const reiser4_key * key) {
-		int ld;
-
-		 assert("nikita-1716", node != NULL);
-		 assert("nikita-1758", key != NULL);
-
-		 RLOCK_DK(znode_get_tree(node));
-		 assert("nikita-1759", znode_contains_key(node, key));
-		 ld = keyeq(znode_get_ld_key(node), key);
-		 RUNLOCK_DK(znode_get_tree(node));
-		 return ld;
-	}
 	assert("nikita-379", h != NULL);
 
 	active = h->active_lh->node;