ref-filter: drop unused buf/sz pairs

The grab_tag_values() and grab_commit_values() functions take both the
"struct object" as well as the buf/sz pair for the actual object bytes.
However, neither function uses the latter, as they pull the data
directly from the parsed object struct.

Let's get rid of these misleading parameters.

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>

diff --git a/ref-filter.c b/ref-filter.c
index 422a9c9ae3fd2cfd552ce48c5e68ce18919b1eb9..c9333b31817348a09b5a695ac78e11513970c7d8 100644 (file)
--- a/ref-filter.c
+++ b/ref-filter.c
@@ -913,7 +913,7 @@ static void grab_common_values(struct atom_value *val, int deref, struct expand_
 }
 
 /* See grab_values */
 }
 
 /* See grab_values */

-static void grab_tag_values(struct atom_value *val, int deref, struct object *obj, void *buf, unsigned long sz)
+static void grab_tag_values(struct atom_value *val, int deref, struct object *obj)

 {
        int i;
        struct tag *tag = (struct tag *) obj;
 {
        int i;
        struct tag *tag = (struct tag *) obj;


@@ -935,7 +935,7 @@ static void grab_tag_values(struct atom_value *val, int deref, struct object *ob
 }
 
 /* See grab_values */
 }
 
 /* See grab_values */

-static void grab_commit_values(struct atom_value *val, int deref, struct object *obj, void *buf, unsigned long sz)
+static void grab_commit_values(struct atom_value *val, int deref, struct object *obj)

 {
        int i;
        struct commit *commit = (struct commit *) obj;
 {
        int i;
        struct commit *commit = (struct commit *) obj;


@@ -1269,12 +1269,12 @@ static void grab_values(struct atom_value *val, int deref, struct object *obj, v
 {
        switch (obj->type) {
        case OBJ_TAG:
 {
        switch (obj->type) {
        case OBJ_TAG:

-               grab_tag_values(val, deref, obj, buf, sz);
+               grab_tag_values(val, deref, obj);

                grab_sub_body_contents(val, deref, obj, buf, sz);
                grab_person("tagger", val, deref, obj, buf, sz);
                break;
        case OBJ_COMMIT:
                grab_sub_body_contents(val, deref, obj, buf, sz);
                grab_person("tagger", val, deref, obj, buf, sz);
                break;
        case OBJ_COMMIT:

-               grab_commit_values(val, deref, obj, buf, sz);
+               grab_commit_values(val, deref, obj);

                grab_sub_body_contents(val, deref, obj, buf, sz);
                grab_person("author", val, deref, obj, buf, sz);
                grab_person("committer", val, deref, obj, buf, sz);
                grab_sub_body_contents(val, deref, obj, buf, sz);
                grab_person("author", val, deref, obj, buf, sz);
                grab_person("committer", val, deref, obj, buf, sz);