From 712154704964906a59d481dc7edf43554f9eaf83 Mon Sep 17 00:00:00 2001 From: Jozsef Kadlecsik Date: Tue, 18 Jan 2011 17:48:22 +0100 Subject: Fix build with NDEBUG defined The usage of the gcc option -Wunused-parameter interferes badly with the assert() macros. In case -DNDEBUG is specified build fails with: cc1: warnings being treated as errors print.c: In function 'ipset_print_family': print.c:92: error: unused parameter 'opt' print.c: In function 'ipset_print_port': print.c:413: error: unused parameter 'opt' print.c: In function 'ipset_print_proto': Fix it by taking into accout NDEBUG in the function arguments. Bug reported by Holger Eitzenberger. --- lib/print.c | 18 ++++++++++++------ 1 file changed, 12 insertions(+), 6 deletions(-) (limited to 'lib/print.c') diff --git a/lib/print.c b/lib/print.c index b6819e5..7ec786f 100644 --- a/lib/print.c +++ b/lib/print.c @@ -89,7 +89,8 @@ ipset_print_ether(char *buf, unsigned int len, */ int ipset_print_family(char *buf, unsigned int len, - const struct ipset_data *data, enum ipset_opt opt, + const struct ipset_data *data, + enum ipset_opt opt ASSERT_UNUSED, uint8_t env UNUSED) { uint8_t family; @@ -410,7 +411,8 @@ ipset_print_name(char *buf, unsigned int len, */ int ipset_print_port(char *buf, unsigned int len, - const struct ipset_data *data, enum ipset_opt opt, + const struct ipset_data *data, + enum ipset_opt opt ASSERT_UNUSED, uint8_t env UNUSED) { const uint16_t *port; @@ -454,7 +456,8 @@ ipset_print_port(char *buf, unsigned int len, */ int ipset_print_proto(char *buf, unsigned int len, - const struct ipset_data *data, enum ipset_opt opt, + const struct ipset_data *data, + enum ipset_opt opt ASSERT_UNUSED, uint8_t env UNUSED) { const struct protoent *protoent; @@ -490,7 +493,8 @@ ipset_print_proto(char *buf, unsigned int len, */ int ipset_print_icmp(char *buf, unsigned int len, - const struct ipset_data *data, enum ipset_opt opt, + const struct ipset_data *data, + enum ipset_opt opt ASSERT_UNUSED, uint8_t env UNUSED) { const char *name; @@ -523,7 +527,8 @@ ipset_print_icmp(char *buf, unsigned int len, */ int ipset_print_icmpv6(char *buf, unsigned int len, - const struct ipset_data *data, enum ipset_opt opt, + const struct ipset_data *data, + enum ipset_opt opt ASSERT_UNUSED, uint8_t env UNUSED) { const char *name; @@ -556,7 +561,8 @@ ipset_print_icmpv6(char *buf, unsigned int len, */ int ipset_print_proto_port(char *buf, unsigned int len, - const struct ipset_data *data, enum ipset_opt opt, + const struct ipset_data *data, + enum ipset_opt opt ASSERT_UNUSED, uint8_t env UNUSED) { int size, offset = 0; -- cgit v1.2.3