They are not part of the libnftables library API, they are not public symbols,
so it doesn't not make sense to have them there. Move the two functions to a
different source file so libnftables.c only has the API functions.

I think copyright belongs to Phil Sutter since he introduced this code back in
commit 2535ba7006f22a6470f4c88ea7d30c343a1d8799 (src: get rid of printf).

Signed-off-by: Arturo Borrero Gonzalez <art...@netfilter.org>
---
v2: move code to src/print.c per Pablo's suggestion.

 src/Makefile.am   |    1 +
 src/libnftables.c |   27 ---------------------------
 src/print.c       |   38 ++++++++++++++++++++++++++++++++++++++
 3 files changed, 39 insertions(+), 27 deletions(-)
 create mode 100644 src/print.c

diff --git a/src/Makefile.am b/src/Makefile.am
index fd64175..a1c18fe 100644
--- a/src/Makefile.am
+++ b/src/Makefile.am
@@ -62,6 +62,7 @@ libnftables_la_SOURCES =                      \
                nfnl_osf.c                      \
                tcpopt.c                        \
                socket.c                        \
+               print.c                         \
                libnftables.c
 
 # yacc and lex generate dirty code
diff --git a/src/libnftables.c b/src/libnftables.c
index dccb8ab..f2cd267 100644
--- a/src/libnftables.c
+++ b/src/libnftables.c
@@ -507,30 +507,3 @@ err:
                cache_release(&nft->cache);
        return rc;
 }
-
-int nft_print(struct output_ctx *octx, const char *fmt, ...)
-{
-       int ret;
-       va_list arg;
-
-       va_start(arg, fmt);
-       ret = vfprintf(octx->output_fp, fmt, arg);
-       va_end(arg);
-       fflush(octx->output_fp);
-
-       return ret;
-}
-
-int nft_gmp_print(struct output_ctx *octx, const char *fmt, ...)
-{
-       int ret;
-       va_list arg;
-
-       va_start(arg, fmt);
-       ret = gmp_vfprintf(octx->output_fp, fmt, arg);
-       va_end(arg);
-       fflush(octx->output_fp);
-
-       return ret;
-}
-
diff --git a/src/print.c b/src/print.c
new file mode 100644
index 0000000..d1b25e8
--- /dev/null
+++ b/src/print.c
@@ -0,0 +1,38 @@
+/*
+ * Copyright (c) 2017 Phil Sutter <p...@nwl.cc>
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License version 2 as
+ * published by the Free Software Foundation.
+ *
+ */
+
+#include <stdarg.h>
+#include <nftables.h>
+#include <utils.h>
+
+int nft_print(struct output_ctx *octx, const char *fmt, ...)
+{
+       int ret;
+       va_list arg;
+
+       va_start(arg, fmt);
+       ret = vfprintf(octx->output_fp, fmt, arg);
+       va_end(arg);
+       fflush(octx->output_fp);
+
+       return ret;
+}
+
+int nft_gmp_print(struct output_ctx *octx, const char *fmt, ...)
+{
+       int ret;
+       va_list arg;
+
+       va_start(arg, fmt);
+       ret = gmp_vfprintf(octx->output_fp, fmt, arg);
+       va_end(arg);
+       fflush(octx->output_fp);
+
+       return ret;
+}

Reply via email to