[MERGED] libosmocore[master]: [doc] Don't add libosmovty bits to libosmocore groups

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

Harald Welte gerrit-no-reply at lists.osmocom.org
Mon Oct 16 20:49:11 UTC 2017


Harald Welte has submitted this change and it was merged.

Change subject: [doc] Don't add libosmovty bits to libosmocore groups
......................................................................


[doc] Don't add libosmovty bits to libosmocore groups

Let's not put files of libosmovty into Doxygen groups of libosmocore,
as this seems to confuse Doxygen.  Also, some minor updates/fixes
of libosmovty documentation.

Change-Id: I70e612b8d06aabefe634fcd7861641ffb941d974
---
M src/vty/fsm_vty.c
M src/vty/logging_vty.c
M src/vty/stats_vty.c
M src/vty/telnet_interface.c
4 files changed, 29 insertions(+), 30 deletions(-)

Approvals:
  Harald Welte: Looks good to me, approved
  Jenkins Builder: Verified



diff --git a/src/vty/fsm_vty.c b/src/vty/fsm_vty.c
index 345be66..50c7fd5 100644
--- a/src/vty/fsm_vty.c
+++ b/src/vty/fsm_vty.c
@@ -1,5 +1,3 @@
-/*! \file fsm_vty.c
- * Osmocom FSM introspection via VTY. */
 /*
  * (C) 2016 by Harald Welte <laforge at gnumonks.org>
  * All Rights Reserved
@@ -35,9 +33,8 @@
 #include <osmocom/core/logging.h>
 #include <osmocom/core/linuxlist.h>
 
-/*! \addtogroup fsm
- *  @{
- *  VTY interface for Osmocom FSM
+/*! \file fsm_vty.c
+ *  Osmocom FSM introspection via VTY.
  *
  *  This is code implementing generic VTY access to Osmocom FSMs from
  *  libosmocore.  This means that any application can expose all state
@@ -187,5 +184,3 @@
 	install_element_ve(&show_fsm_inst_cmd);
 	install_element_ve(&show_fsm_insts_cmd);
 }
-
-/*! @} */
diff --git a/src/vty/logging_vty.c b/src/vty/logging_vty.c
index 0ab7686..9b7d10a 100644
--- a/src/vty/logging_vty.c
+++ b/src/vty/logging_vty.c
@@ -1,5 +1,3 @@
-/*! \file logging_vty.c
- * OpenBSC logging helper for the VTY. */
 /*
  * (C) 2009-2010 by Harald Welte <laforge at gnumonks.org>
  * (C) 2009-2014 by Holger Hans Peter Freyther
@@ -41,14 +39,21 @@
 
 #define LOG_STR "Configure logging sub-system\n"
 
-/*! \addtogroup logging
- *  @{
+/*! \file logging_vty.c
  *  Configuration of logging from VTY
  *
- *  This module implements functions that permit configuration of
- *  the libosmocore logging framework from VTY commands.  This applies
- *  both to logging to the VTY (telnet sessions), as well as logging to
- *  other targets, such as sysslog, file, gsmtap, ...
+ *  This module implements
+ *  - functions that permit configuration of the libosmocore logging
+ *    framework from VTY commands in the configure -> logging node.
+ *
+ *  - functions that permit logging *to* a VTY session.  Basically each
+ *    VTY session gets its own log target, with configurable
+ *    per-subsystem log levels.  This is performed internally via the
+ *    \ref log_target_create_vty function.
+ *
+ *  You have to call \ref logging_vty_add_cmds from your application
+ *  once to enable both of the above.
+ *
  */
 
 extern const struct log_info *osmo_log_info;
@@ -756,6 +761,8 @@
 	return 1;
 }
 
+/*! Register logging related commands to the VTY. Call this once from
+ *  your application if you want to support those commands. */
 void logging_vty_add_cmds()
 {
 	install_element_ve(&enable_logging_cmd);
@@ -796,5 +803,3 @@
 #endif
 	install_element(CONFIG_NODE, &cfg_log_gsmtap_cmd);
 }
-
-/* @} */
diff --git a/src/vty/stats_vty.c b/src/vty/stats_vty.c
index 95f04bc..2ccbf66 100644
--- a/src/vty/stats_vty.c
+++ b/src/vty/stats_vty.c
@@ -1,5 +1,3 @@
-/*! \file stats_vty.c
- * OpenBSC stats helper for the VTY. */
 /*
  * (C) 2009-2010 by Harald Welte <laforge at gnumonks.org>
  * (C) 2009-2014 by Holger Hans Peter Freyther
@@ -43,9 +41,14 @@
 
 #define SHOW_STATS_STR "Show statistical values\n"
 
-/*! \addtogroup stats
- *  @{
+/*! \file stats_vty.c
  *  VTY interface for statsd / statistic items
+ *
+ *  This code allows you to register a couple of VTY commands that
+ *  permit configuration of the \ref stats functionality from the VTY.
+ *
+ *  Use \ref osmo_stats_vty_add_cmds once at application start-up to
+ *  enable related commands.
  */
 
 /* containing version info */
@@ -609,5 +612,3 @@
 
 	install_element_ve(&show_stats_asciidoc_table_cmd);
 }
-
-/*! @} */
diff --git a/src/vty/telnet_interface.c b/src/vty/telnet_interface.c
index f82a7ff..40154fe 100644
--- a/src/vty/telnet_interface.c
+++ b/src/vty/telnet_interface.c
@@ -1,5 +1,3 @@
-/*! \file telnet_interface.c
- * minimalistic telnet/network interface it might turn into a wire interface */
 /* (C) 2009 by Holger Hans Peter Freyther <zecke at selfish.org>
  * All Rights Reserved
  *
@@ -37,8 +35,7 @@
 #include <osmocom/vty/buffer.h>
 #include <osmocom/vty/command.h>
 
-/*! \addtogroup telnet_interface
- *  @{
+/*! \file telnet_interface.c
  *  Telnet interface towards Osmocom VTY
  *
  *  This module contains the code implementing a telnet server for VTY
@@ -46,7 +43,9 @@
  *  process in order to enable interactive command-line introspection,
  *  interaction and configuration.
  *
- * \file telnet_interface.c */
+ *  You typically call \ref telnet_init or \ref telnet_init_dynif once
+ *  from your application code to enable this.
+ */
 
 /* per connection data */
 LLIST_HEAD(active_connections);
@@ -219,6 +218,7 @@
 	}
 }
 
+/*! Close all telnet connections and release the telnet socket */
 void telnet_exit(void) 
 {
 	struct telnet_connection *tc, *tc2;
@@ -230,5 +230,3 @@
 	close(server_socket.fd);
 	talloc_free(tall_telnet_ctx);
 }
-
-/*! @} */

-- 
To view, visit https://gerrit.osmocom.org/4293
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I70e612b8d06aabefe634fcd7861641ffb941d974
Gerrit-PatchSet: 1
Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Owner: Harald Welte <laforge at gnumonks.org>
Gerrit-Reviewer: Harald Welte <laforge at gnumonks.org>
Gerrit-Reviewer: Jenkins Builder



More information about the gerrit-log mailing list