ctdb-common: Return value of ctdb_queue_length() should be unsigned

Compiling with -Wsign-compare complains:

ctdb/server/ctdb_daemon.c: scope_hint: In function ‘daemon_queue_send’
ctdb/server/ctdb_daemon.c:259:40: warning: comparison of integer expressions of different signedness: ‘int’ and ‘uint32_t’ {aka ‘unsigned int’} [-Wsign-compare]
...

The struct ctdb_queue member out_queue_length is actually uint32_t, so
just return that type.

Found by csbuild.

Signed-off-by: Martin Schwenke <martin@meltin.net>
Reviewed-by: Amitay Isaacs <amitay@gmail.com>

diff --git a/ctdb/common/common.h b/ctdb/common/common.h
index 79f6b9ed03a51083ab75815f18d8dac75c0450dc..c50b52a5eb5092a33bea15b8ba1e4b4f4d5b47a2 100644 (file)
--- a/ctdb/common/common.h
+++ b/ctdb/common/common.h
@@ -27,7 +27,7 @@
 typedef void (*ctdb_queue_cb_fn_t)(uint8_t *data, size_t length,
                                   void *private_data);
 
-int ctdb_queue_length(struct ctdb_queue *queue);
+uint32_t ctdb_queue_length(struct ctdb_queue *queue);
 
 int ctdb_queue_send(struct ctdb_queue *queue, uint8_t *data, uint32_t length);
 

diff --git a/ctdb/common/ctdb_io.c b/ctdb/common/ctdb_io.c
index c16eb7f67b7a671756b7402cba4229a20d4e0652..000f71e4c201a24329e12ad28c26d606443cc10c 100644 (file)
--- a/ctdb/common/ctdb_io.c
+++ b/ctdb/common/ctdb_io.c
@@ -72,7 +72,7 @@ struct ctdb_queue {
 
 
 
-int ctdb_queue_length(struct ctdb_queue *queue)
+uint32_t ctdb_queue_length(struct ctdb_queue *queue)
 {
        return queue->out_queue_length;
 }