osmith submitted this change.

View Change


Approvals: pespin: Looks good to me, but someone else must approve Jenkins Builder: Verified fixeria: Looks good to me, approved
gtp/gsn_internal.h: new file

Change-Id: I999462e39411fc4ec7e50bd0212e870006fbc4f1
---
M gtp/Makefile.am
M gtp/gsn.c
A gtp/gsn_internal.h
M gtp/gtp.c
M include/osmocom/gtp/gsn.h
5 files changed, 15 insertions(+), 3 deletions(-)

diff --git a/gtp/Makefile.am b/gtp/Makefile.am
index f0c7555..11426a4 100644
--- a/gtp/Makefile.am
+++ b/gtp/Makefile.am
@@ -16,6 +16,7 @@

libgtp_la_SOURCES = \
gsn.c \
+ gsn_internal.h \
gtp.c \
gtpie.c \
lookupa.c \
diff --git a/gtp/gsn.c b/gtp/gsn.c
index e326f80..c23fb2a 100644
--- a/gtp/gsn.c
+++ b/gtp/gsn.c
@@ -62,6 +62,7 @@
#include <osmocom/gtp/gtpie.h>

#include "queue.h"
+#include "gsn_internal.h"

/* Error reporting functions */

diff --git a/gtp/gsn_internal.h b/gtp/gsn_internal.h
new file mode 100644
index 0000000..deb60c3
--- /dev/null
+++ b/gtp/gsn_internal.h
@@ -0,0 +1,3 @@
+#pragma once
+
+void gtp_queue_timer_start(struct gsn_t *gsn);
diff --git a/gtp/gtp.c b/gtp/gtp.c
index 8921802..bdf7e7d 100644
--- a/gtp/gtp.c
+++ b/gtp/gtp.c
@@ -60,6 +60,7 @@
#include <osmocom/gtp/gtpie.h>

#include "queue.h"
+#include "gsn_internal.h"

/* Error reporting functions */

diff --git a/include/osmocom/gtp/gsn.h b/include/osmocom/gtp/gsn.h
index 8e47ce3..936db9b 100644
--- a/include/osmocom/gtp/gsn.h
+++ b/include/osmocom/gtp/gsn.h
@@ -175,7 +175,4 @@
extern int gtp_retrans(struct gsn_t *gsn) OSMO_DEPRECATED("This API is a no-op, libgtp already does the job internally");
extern int gtp_retranstimeout(struct gsn_t *gsn, struct timeval *timeout) OSMO_DEPRECATED("This API is a no-op and will return a 1 day timeout");

-/* Internal APIs: */
-void gtp_queue_timer_start(struct gsn_t *gsn);
-
#endif /* !_GSN_H */

To view, visit change 36067. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: osmo-ggsn
Gerrit-Branch: master
Gerrit-Change-Id: I999462e39411fc4ec7e50bd0212e870006fbc4f1
Gerrit-Change-Number: 36067
Gerrit-PatchSet: 2
Gerrit-Owner: osmith <osmith@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: osmith <osmith@sysmocom.de>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-MessageType: merged