daniel submitted this change.

View Change


Approvals: pespin: Looks good to me, approved Jenkins Builder: Verified
osmo_{client,server}_main: Remove is_config_node in vty_app_info

This member is deprcated/unused so simply remove it:

> 'is_config_node' is deprecated: Implicit parent node tracking has
> replaced the use of this callback. This callback is no longer called,
> ever, and can be left NULL.

Change-Id: I34295950557d9009b0d8d3378deb3b1263476f82
---
M src/osmo_client_main.c
M src/osmo_common.c
M src/osmo_server_main.c
3 files changed, 15 insertions(+), 12 deletions(-)

diff --git a/src/osmo_client_main.c b/src/osmo_client_main.c
index 8ca2df3..818a2f1 100644
--- a/src/osmo_client_main.c
+++ b/src/osmo_client_main.c
@@ -79,7 +79,6 @@
.name = "OsmoPCAPClient",
.version = PACKAGE_VERSION,
.go_parent_cb = osmopcap_go_parent,
- .is_config_node = osmopcap_is_config_node,
};

static void print_usage()
diff --git a/src/osmo_common.c b/src/osmo_common.c
index 39f448a..7c641e6 100644
--- a/src/osmo_common.c
+++ b/src/osmo_common.c
@@ -88,13 +88,3 @@

return vty->node;
}
-
-int osmopcap_is_config_node(struct vty *vty, int node)
-{
- switch (node) {
- case CONFIG_NODE:
- return 0;
- default:
- return 1;
- }
-}
diff --git a/src/osmo_server_main.c b/src/osmo_server_main.c
index 8ad89ba..4633888 100644
--- a/src/osmo_server_main.c
+++ b/src/osmo_server_main.c
@@ -89,7 +89,6 @@
.name = "OsmoPCAPServer",
.version = PACKAGE_VERSION,
.go_parent_cb = osmopcap_go_parent,
- .is_config_node = osmopcap_is_config_node,
};

static void print_usage()

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

Gerrit-Project: osmo-pcap
Gerrit-Branch: master
Gerrit-Change-Id: I34295950557d9009b0d8d3378deb3b1263476f82
Gerrit-Change-Number: 31588
Gerrit-PatchSet: 2
Gerrit-Owner: daniel <dwillmann@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: daniel <dwillmann@sysmocom.de>
Gerrit-Reviewer: osmith <osmith@sysmocom.de>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-MessageType: merged