From 3749dc93a3e0ab9a2e4c19f42f7879a80265621e Mon Sep 17 00:00:00 2001 From: Holger Hans Peter Freyther Date: Wed, 9 Jan 2013 16:54:49 +0100 Subject: ctrl: Fix the signature of the string control commands Like with all type unsafe callbacks we will need to cast from void to the dtype. This addresses some compiler warnings. Make it possible to only include the control_cmd.h to use the macros defined in this file. --- openbsc/include/openbsc/control_cmd.h | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/openbsc/include/openbsc/control_cmd.h b/openbsc/include/openbsc/control_cmd.h index 3dc5b546a..57785b87f 100644 --- a/openbsc/include/openbsc/control_cmd.h +++ b/openbsc/include/openbsc/control_cmd.h @@ -2,10 +2,13 @@ #define _CONTROL_CMD_H #include +#include #include #include +#include + #define CTRL_CMD_ERROR -1 #define CTRL_CMD_HANDLED 0 #define CTRL_CMD_REPLY 1 @@ -118,8 +121,9 @@ struct ctrl_cmd_element cmd_##cmdname = { \ } #define CTRL_CMD_DEFINE_STRING(cmdname, cmdstr, dtype, element) \ -static int get_##cmdname(struct ctrl_cmd *cmd, dtype *data) \ +static int get_##cmdname(struct ctrl_cmd *cmd, void *_data) \ { \ + dtype *data = _data; \ cmd->reply = talloc_asprintf(cmd, "%s", data->element); \ if (!cmd->reply) { \ cmd->reply = "OOM"; \ @@ -127,8 +131,9 @@ static int get_##cmdname(struct ctrl_cmd *cmd, dtype *data) \ } \ return CTRL_CMD_REPLY; \ } \ -static int set_##cmdname(struct ctrl_cmd *cmd, dtype *data) \ +static int set_##cmdname(struct ctrl_cmd *cmd, void *_data) \ { \ + dtype *data = _data; \ bsc_replace_string(cmd->node, &data->element, cmd->value); \ return get_##cmdname(cmd, data); \ } \ -- cgit v1.2.3