diff options
author | Holger Hans Peter Freyther <holger@moiji-mobile.com> | 2015-08-02 02:14:07 +0000 |
---|---|---|
committer | Holger Hans Peter Freyther <holger@moiji-mobile.com> | 2015-08-05 04:22:56 +0000 |
commit | a9e5252128d1d871739f067fe9b0ca9b9a75a6f7 (patch) | |
tree | 7f92975a8ef4883b3ca5891bb2a03582270484ca /include/osmocom/vty/vty.h | |
parent | c136da587fc2fafff1f649828eae5c996c345017 (diff) |
vty: Change API to have node installation be done by int
We are mixing enums and hope that no short-enums are used. This
is leading to a lot compiler warnings generated by clang. Change
the API to work with integers.
Porting:
The go_parent_cb implementations in the applications need to be
fixed. The API change leads to a compile time warning.
Fixes:
abis_om2000_vty.c:46:2: warning: implicit conversion from enumeration type 'enum bsc_vty_node' to
different enumeration type 'enum node_type' [-Wenum-conversion]
OM2K_NODE,
^~~~~~~~~
Diffstat (limited to 'include/osmocom/vty/vty.h')
-rw-r--r-- | include/osmocom/vty/vty.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/include/osmocom/vty/vty.h b/include/osmocom/vty/vty.h index 1dcc2300..36843974 100644 --- a/include/osmocom/vty/vty.h +++ b/include/osmocom/vty/vty.h @@ -156,7 +156,7 @@ struct vty_app_info { /*! \brief \ref talloc context */ void *tall_ctx; /*! \brief call-back for returning to parent n ode */ - enum node_type (*go_parent_cb)(struct vty *vty); + int (*go_parent_cb)(struct vty *vty); /*! \brief call-back to determine if node is config node */ int (*is_config_node)(struct vty *vty, int node); /*! \brief Check if the config is consistent before write */ @@ -184,7 +184,7 @@ int vty_shell_serv (struct vty *); void vty_hello (struct vty *); void *vty_current_index(struct vty *); int vty_current_node(struct vty *vty); -enum node_type vty_go_parent(struct vty *vty); +int vty_go_parent(struct vty *vty); extern void *tall_vty_ctx; |