fsck: mark unused parameters in various fsck callbacks

There are a few callback functions which are used with the fsck code,
but it's natural that not all callbacks need all parameters. For
reporting, even something as obvious as "the oid of the object which had
a problem" is not always used, as some callers are only checking a
single object in the first place. And for both reporting and walking,
things like void data pointers and the fsck_options aren't always
necessary.

But since each such parameter is used by _some_ callback, we have to
keep them in the interface. Mark the unused ones in specific callbacks
to avoid triggering -Wunused-parameter.

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

diff --git a/builtin/fsck.c b/builtin/fsck.c
index 3aa9c812eb5443b2f294689eb039ab3752831c4e..2a494d61973233ca05b46798f0543ba6444b79f3 100644 (file)
--- a/builtin/fsck.c
+++ b/builtin/fsck.c
@@ -92,11 +92,11 @@ static int objerror(struct object *obj, const char *err)
        return -1;
 }
 
-static int fsck_error_func(struct fsck_options *o,
+static int fsck_error_func(struct fsck_options *o UNUSED,
                           const struct object_id *oid,
                           enum object_type object_type,
                           enum fsck_msg_type msg_type,
-                          enum fsck_msg_id msg_id,
+                          enum fsck_msg_id msg_id UNUSED,
                           const char *message)
 {
        switch (msg_type) {
@@ -121,7 +121,7 @@ static int fsck_error_func(struct fsck_options *o,
 static struct object_array pending;
 
 static int mark_object(struct object *obj, enum object_type type,
-                      void *data, struct fsck_options *options)
+                      void *data, struct fsck_options *options UNUSED)
 {
        struct object *parent = data;
 
@@ -206,8 +206,8 @@ static int traverse_reachable(void)
        return !!result;
 }
 
-static int mark_used(struct object *obj, enum object_type object_type,
-                    void *data, struct fsck_options *options)
+static int mark_used(struct object *obj, int type UNUSED,
+                    void *data UNUSED, struct fsck_options *options UNUSED)
 {
        if (!obj)
                return 1;

diff --git a/builtin/index-pack.c b/builtin/index-pack.c
index e280180cac1b2c9255f92f4613fe8eacebb5c653..3f74364a682b90a1b370b86897ebca9079987ca4 100644 (file)
--- a/builtin/index-pack.c
+++ b/builtin/index-pack.c
@@ -223,7 +223,8 @@ static void cleanup_thread(void)
 }
 
 static int mark_link(struct object *obj, enum object_type type,
-                    void *data, struct fsck_options *options)
+                    void *data UNUSED,
+                    struct fsck_options *options UNUSED)
 {
        if (!obj)
                return -1;

diff --git a/builtin/mktag.c b/builtin/mktag.c
index 43e2766db4934e371659068952c8d4a4705aaa53..d8e0b5afc079d2f71383eed6881b733fea251312 100644 (file)
--- a/builtin/mktag.c
+++ b/builtin/mktag.c
@@ -18,11 +18,11 @@ static int option_strict = 1;
 
 static struct fsck_options fsck_options = FSCK_OPTIONS_STRICT;
 
-static int mktag_fsck_error_func(struct fsck_options *o,
-                                const struct object_id *oid,
-                                enum object_type object_type,
+static int mktag_fsck_error_func(struct fsck_options *o UNUSED,
+                                const struct object_id *oid UNUSED,
+                                enum object_type object_type UNUSED,
                                 enum fsck_msg_type msg_type,
-                                enum fsck_msg_id msg_id,
+                                enum fsck_msg_id msg_id UNUSED,
                                 const char *message)
 {
        switch (msg_type) {

diff --git a/builtin/unpack-objects.c b/builtin/unpack-objects.c
index 1979532a9df0940581129d8c4279b8126f53b715..558b8485ba2ff1b07280c0f9efbf963e326a48fb 100644 (file)
--- a/builtin/unpack-objects.c
+++ b/builtin/unpack-objects.c
@@ -214,7 +214,8 @@ static void write_cached_object(struct object *obj, struct obj_buffer *obj_buf)
  * Verify its reachability and validity recursively and write it out.
  */
 static int check_object(struct object *obj, enum object_type type,
-                       void *data, struct fsck_options *options)
+                       void *data UNUSED,
+                       struct fsck_options *options UNUSED)
 {
        struct obj_buffer *obj_buf;
 

diff --git a/fsck.c b/fsck.c
index 3be86616c51b66f6f677c6d87a7a163a3e908537..8852e56f6b6ae0907035de9d3e132a6484c12c1c 100644 (file)
--- a/fsck.c
+++ b/fsck.c
@@ -1310,9 +1310,9 @@ int fsck_buffer(const struct object_id *oid, enum object_type type,
 
 int fsck_error_function(struct fsck_options *o,
                        const struct object_id *oid,
-                       enum object_type object_type,
+                       enum object_type object_type UNUSED,
                        enum fsck_msg_type msg_type,
-                       enum fsck_msg_id msg_id,
+                       enum fsck_msg_id msg_id UNUSED,
                        const char *message)
 {
        if (msg_type == FSCK_WARN) {

diff --git a/object-file.c b/object-file.c
index 8d87720dd53f851a4ef52e7f96f5202fd2a0840f..68a2397e33a00cd3ef60ad6f138dc69e69c2225b 100644 (file)
--- a/object-file.c
+++ b/object-file.c
@@ -2308,11 +2308,11 @@ int repo_has_object_file(struct repository *r,
  * report the minimal fsck error here, and rely on the caller to
  * give more context.
  */
-static int hash_format_check_report(struct fsck_options *opts,
-                                    const struct object_id *oid,
-                                    enum object_type object_type,
-                                    enum fsck_msg_type msg_type,
-                                    enum fsck_msg_id msg_id,
+static int hash_format_check_report(struct fsck_options *opts UNUSED,
+                                    const struct object_id *oid UNUSED,
+                                    enum object_type object_type UNUSED,
+                                    enum fsck_msg_type msg_type UNUSED,
+                                    enum fsck_msg_id msg_id UNUSED,
                                     const char *message)
 {
        error(_("object fails fsck: %s"), message);