]> git.ipfire.org Git - thirdparty/git.git/blobdiff - notes.c
notes.h/c: Allow combine_notes functions to remove notes
[thirdparty/git.git] / notes.c
diff --git a/notes.c b/notes.c
index 197824439826881fc0de3275353ec1b74d86cfa3..0c13a36eb7d39be92743c2906412412b73ccf8f1 100644 (file)
--- a/notes.c
+++ b/notes.c
@@ -149,6 +149,79 @@ static struct leaf_node *note_tree_find(struct notes_tree *t,
        return NULL;
 }
 
+/*
+ * How to consolidate an int_node:
+ * If there are > 1 non-NULL entries, give up and return non-zero.
+ * Otherwise replace the int_node at the given index in the given parent node
+ * with the only entry (or a NULL entry if no entries) from the given tree,
+ * and return 0.
+ */
+static int note_tree_consolidate(struct int_node *tree,
+       struct int_node *parent, unsigned char index)
+{
+       unsigned int i;
+       void *p = NULL;
+
+       assert(tree && parent);
+       assert(CLR_PTR_TYPE(parent->a[index]) == tree);
+
+       for (i = 0; i < 16; i++) {
+               if (GET_PTR_TYPE(tree->a[i]) != PTR_TYPE_NULL) {
+                       if (p) /* more than one entry */
+                               return -2;
+                       p = tree->a[i];
+               }
+       }
+
+       /* replace tree with p in parent[index] */
+       parent->a[index] = p;
+       free(tree);
+       return 0;
+}
+
+/*
+ * To remove a leaf_node:
+ * Search to the tree location appropriate for the given leaf_node's key:
+ * - If location does not hold a matching entry, abort and do nothing.
+ * - Replace the matching leaf_node with a NULL entry (and free the leaf_node).
+ * - Consolidate int_nodes repeatedly, while walking up the tree towards root.
+ */
+static void note_tree_remove(struct notes_tree *t, struct int_node *tree,
+               unsigned char n, struct leaf_node *entry)
+{
+       struct leaf_node *l;
+       struct int_node *parent_stack[20];
+       unsigned char i, j;
+       void **p = note_tree_search(t, &tree, &n, entry->key_sha1);
+
+       assert(GET_PTR_TYPE(entry) == 0); /* no type bits set */
+       if (GET_PTR_TYPE(*p) != PTR_TYPE_NOTE)
+               return; /* type mismatch, nothing to remove */
+       l = (struct leaf_node *) CLR_PTR_TYPE(*p);
+       if (hashcmp(l->key_sha1, entry->key_sha1))
+               return; /* key mismatch, nothing to remove */
+
+       /* we have found a matching entry */
+       free(l);
+       *p = SET_PTR_TYPE(NULL, PTR_TYPE_NULL);
+
+       /* consolidate this tree level, and parent levels, if possible */
+       if (!n)
+               return; /* cannot consolidate top level */
+       /* first, build stack of ancestors between root and current node */
+       parent_stack[0] = t->root;
+       for (i = 0; i < n; i++) {
+               j = GET_NIBBLE(i, entry->key_sha1);
+               parent_stack[i + 1] = CLR_PTR_TYPE(parent_stack[i]->a[j]);
+       }
+       assert(i == n && parent_stack[i] == tree);
+       /* next, unwind stack until note_tree_consolidate() is done */
+       while (i > 0 &&
+              !note_tree_consolidate(parent_stack[i], parent_stack[i - 1],
+                                     GET_NIBBLE(i - 1, entry->key_sha1)))
+               i--;
+}
+
 /*
  * To insert a leaf_node:
  * Search to the tree location appropriate for the given leaf_node's key:
@@ -175,7 +248,10 @@ static void note_tree_insert(struct notes_tree *t, struct int_node *tree,
        switch (GET_PTR_TYPE(*p)) {
        case PTR_TYPE_NULL:
                assert(!*p);
-               *p = SET_PTR_TYPE(entry, type);
+               if (is_null_sha1(entry->val_sha1))
+                       free(entry);
+               else
+                       *p = SET_PTR_TYPE(entry, type);
                return;
        case PTR_TYPE_NOTE:
                switch (type) {
@@ -191,6 +267,9 @@ static void note_tree_insert(struct notes_tree *t, struct int_node *tree,
                                            sha1_to_hex(l->val_sha1),
                                            sha1_to_hex(entry->val_sha1),
                                            sha1_to_hex(l->key_sha1));
+
+                               if (is_null_sha1(l->val_sha1))
+                                       note_tree_remove(t, tree, n, entry);
                                free(entry);
                                return;
                        }
@@ -222,6 +301,10 @@ static void note_tree_insert(struct notes_tree *t, struct int_node *tree,
        /* non-matching leaf_node */
        assert(GET_PTR_TYPE(*p) == PTR_TYPE_NOTE ||
               GET_PTR_TYPE(*p) == PTR_TYPE_SUBTREE);
+       if (is_null_sha1(entry->val_sha1)) { /* skip insertion of empty note */
+               free(entry);
+               return;
+       }
        new_node = (struct int_node *) xcalloc(sizeof(struct int_node), 1);
        note_tree_insert(t, new_node, n + 1, l, GET_PTR_TYPE(*p),
                         combine_notes);
@@ -229,79 +312,6 @@ static void note_tree_insert(struct notes_tree *t, struct int_node *tree,
        note_tree_insert(t, new_node, n + 1, entry, type, combine_notes);
 }
 
-/*
- * How to consolidate an int_node:
- * If there are > 1 non-NULL entries, give up and return non-zero.
- * Otherwise replace the int_node at the given index in the given parent node
- * with the only entry (or a NULL entry if no entries) from the given tree,
- * and return 0.
- */
-static int note_tree_consolidate(struct int_node *tree,
-       struct int_node *parent, unsigned char index)
-{
-       unsigned int i;
-       void *p = NULL;
-
-       assert(tree && parent);
-       assert(CLR_PTR_TYPE(parent->a[index]) == tree);
-
-       for (i = 0; i < 16; i++) {
-               if (GET_PTR_TYPE(tree->a[i]) != PTR_TYPE_NULL) {
-                       if (p) /* more than one entry */
-                               return -2;
-                       p = tree->a[i];
-               }
-       }
-
-       /* replace tree with p in parent[index] */
-       parent->a[index] = p;
-       free(tree);
-       return 0;
-}
-
-/*
- * To remove a leaf_node:
- * Search to the tree location appropriate for the given leaf_node's key:
- * - If location does not hold a matching entry, abort and do nothing.
- * - Replace the matching leaf_node with a NULL entry (and free the leaf_node).
- * - Consolidate int_nodes repeatedly, while walking up the tree towards root.
- */
-static void note_tree_remove(struct notes_tree *t, struct int_node *tree,
-               unsigned char n, struct leaf_node *entry)
-{
-       struct leaf_node *l;
-       struct int_node *parent_stack[20];
-       unsigned char i, j;
-       void **p = note_tree_search(t, &tree, &n, entry->key_sha1);
-
-       assert(GET_PTR_TYPE(entry) == 0); /* no type bits set */
-       if (GET_PTR_TYPE(*p) != PTR_TYPE_NOTE)
-               return; /* type mismatch, nothing to remove */
-       l = (struct leaf_node *) CLR_PTR_TYPE(*p);
-       if (hashcmp(l->key_sha1, entry->key_sha1))
-               return; /* key mismatch, nothing to remove */
-
-       /* we have found a matching entry */
-       free(l);
-       *p = SET_PTR_TYPE(NULL, PTR_TYPE_NULL);
-
-       /* consolidate this tree level, and parent levels, if possible */
-       if (!n)
-               return; /* cannot consolidate top level */
-       /* first, build stack of ancestors between root and current node */
-       parent_stack[0] = t->root;
-       for (i = 0; i < n; i++) {
-               j = GET_NIBBLE(i, entry->key_sha1);
-               parent_stack[i + 1] = CLR_PTR_TYPE(parent_stack[i]->a[j]);
-       }
-       assert(i == n && parent_stack[i] == tree);
-       /* next, unwind stack until note_tree_consolidate() is done */
-       while (i > 0 &&
-              !note_tree_consolidate(parent_stack[i], parent_stack[i - 1],
-                                     GET_NIBBLE(i - 1, entry->key_sha1)))
-               i--;
-}
-
 /* Free the entire notes data contained in the given tree */
 static void note_tree_free(struct int_node *tree)
 {
@@ -898,7 +908,7 @@ static int notes_display_config(const char *k, const char *v, void *cb)
        return 0;
 }
 
-static const char *default_notes_ref(void)
+const char *default_notes_ref(void)
 {
        const char *notes_ref = NULL;
        if (!notes_ref)
@@ -940,7 +950,7 @@ void init_notes(struct notes_tree *t, const char *notes_ref,
                return;
        if (get_tree_entry(object_sha1, "", sha1, &mode))
                die("Failed to read notes tree referenced by %s (%s)",
-                   notes_ref, object_sha1);
+                   notes_ref, sha1_to_hex(object_sha1));
 
        hashclr(root_tree.key_sha1);
        hashcpy(root_tree.val_sha1, sha1);