Mercurial > dovecot > core-2.2
changeset 19863:516e7c50d32f
doveadm: Code cleanup - coding style fixes for doveadm_cmd_param*() accessors
author | Aki Tuomi <aki.tuomi@dovecot.fi> |
---|---|
date | Fri, 26 Feb 2016 09:37:06 +0200 |
parents | b52e1ad3b4f5 |
children | 1d53dbca22da |
files | src/doveadm/doveadm-cmd.c src/doveadm/doveadm-cmd.h |
diffstat | 2 files changed, 38 insertions(+), 25 deletions(-) [+] |
line wrap: on
line diff
--- a/src/doveadm/doveadm-cmd.c Fri Feb 26 09:33:25 2016 +0200 +++ b/src/doveadm/doveadm-cmd.c Fri Feb 26 09:37:06 2016 +0200 @@ -186,7 +186,8 @@ } static const struct doveadm_cmd_param* -doveadm_cmd_param_get(int argc, const struct doveadm_cmd_param* params, const char *name) +doveadm_cmd_param_get(int argc, const struct doveadm_cmd_param *params, + const char *name) { i_assert(params != NULL); for(int i = 0; i < argc; i++) { @@ -196,72 +197,78 @@ return NULL; } -bool doveadm_cmd_param_bool(int argc, const struct doveadm_cmd_param* params, const char *name, bool* value) +bool doveadm_cmd_param_bool(int argc, const struct doveadm_cmd_param *params, + const char *name, bool *value_r) { - const struct doveadm_cmd_param* param; + const struct doveadm_cmd_param *param; if ((param = doveadm_cmd_param_get(argc, params, name))==NULL) return FALSE; if (param->type == CMD_PARAM_BOOL) { - *value = param->value.v_bool; + *value_r = param->value.v_bool; return TRUE; } return FALSE; } -bool doveadm_cmd_param_int64(int argc, const struct doveadm_cmd_param* params, const char *name, int64_t* value) +bool doveadm_cmd_param_int64(int argc, const struct doveadm_cmd_param *params, + const char *name, int64_t *value_r) { - const struct doveadm_cmd_param* param; + const struct doveadm_cmd_param *param; if ((param = doveadm_cmd_param_get(argc, params, name))==NULL) return FALSE; if (param->type == CMD_PARAM_INT64) { - *value = param->value.v_int64; + *value_r = param->value.v_int64; return TRUE; } return FALSE; } -bool doveadm_cmd_param_str(int argc, const struct doveadm_cmd_param* params, const char *name, const char** value) +bool doveadm_cmd_param_str(int argc, const struct doveadm_cmd_param *params, + const char *name, const char **value_r) { - const struct doveadm_cmd_param* param; + const struct doveadm_cmd_param *param; if ((param = doveadm_cmd_param_get(argc, params, name))==NULL) return FALSE; if (param->type == CMD_PARAM_STR) { - *value = param->value.v_string; + *value_r = param->value.v_string; return TRUE; } return FALSE; } -bool doveadm_cmd_param_ip(int argc, const struct doveadm_cmd_param* params, const char *name, struct ip* value) +bool doveadm_cmd_param_ip(int argc, const struct doveadm_cmd_param *params, + const char *name, struct ip_addr *value_r) { - const struct doveadm_cmd_param* param; + const struct doveadm_cmd_param *param; if ((param = doveadm_cmd_param_get(argc, params, name))==NULL) return FALSE; if (param->type == CMD_PARAM_IP) { - memcpy(value, ¶m->value.v_ip, sizeof(struct ip_addr)); + memcpy(value_r, ¶m->value.v_ip, sizeof(struct ip_addr)); return TRUE; } return FALSE; } -bool doveadm_cmd_param_array(int argc, const struct doveadm_cmd_param* params, const char *name, ARRAY_TYPE(const_string)** value) +bool doveadm_cmd_param_array(int argc, const struct doveadm_cmd_param *params, + const char *name, ARRAY_TYPE(const_string) **value_r) { - const struct doveadm_cmd_param* param; + const struct doveadm_cmd_param *param; if ((param = doveadm_cmd_param_get(argc, params, name))==NULL) return FALSE; if (param->type == CMD_PARAM_ARRAY) { - *value = (ARRAY_TYPE(const_string)*)&(param->value.v_array); + *value_r = (ARRAY_TYPE(const_string)*)&(param->value.v_array); return TRUE; } return FALSE; } -bool doveadm_cmd_param_istream(int argc, const struct doveadm_cmd_param* params, const char *name, struct istream** value) +bool doveadm_cmd_param_istream(int argc, const struct doveadm_cmd_param *params, + const char *name, struct istream **value_r) { - const struct doveadm_cmd_param* param; + const struct doveadm_cmd_param *param; if ((param = doveadm_cmd_param_get(argc, params, name))==NULL) return FALSE; if (param->type == CMD_PARAM_ISTREAM) { - *value = param->value.v_istream; + *value_r = param->value.v_istream; return TRUE; } return FALSE;
--- a/src/doveadm/doveadm-cmd.h Fri Feb 26 09:33:25 2016 +0200 +++ b/src/doveadm/doveadm-cmd.h Fri Feb 26 09:37:06 2016 +0200 @@ -116,12 +116,18 @@ int doveadm_cmd_run_ver2(const struct doveadm_cmd_ver2 *cmd, int argc, const char *argv[]); -bool doveadm_cmd_param_bool(int argc, const struct doveadm_cmd_param* params, const char *name, bool* value); -bool doveadm_cmd_param_int64(int argc, const struct doveadm_cmd_param* params, const char *name, int64_t* value); -bool doveadm_cmd_param_str(int argc, const struct doveadm_cmd_param* params, const char *name, const char** value); -bool doveadm_cmd_param_ip(int argc, const struct doveadm_cmd_param* params, const char *name, struct ip_addr *ip); -bool doveadm_cmd_param_array(int argc, const struct doveadm_cmd_param* params, const char *name, ARRAY_TYPE(const_string)** value); -bool doveadm_cmd_param_istream(int argc, const struct doveadm_cmd_param* params, const char *name, struct istream** value); +bool doveadm_cmd_param_bool(int argc, const struct doveadm_cmd_param *params, + const char *name, bool *value_r); +bool doveadm_cmd_param_int64(int argc, const struct doveadm_cmd_param *params, + const char *name, int64_t *value_r); +bool doveadm_cmd_param_str(int argc, const struct doveadm_cmd_param *params, + const char *name, const char **value_r); +bool doveadm_cmd_param_ip(int argc, const struct doveadm_cmd_param *params, + const char *name, struct ip_addr *value_r); +bool doveadm_cmd_param_array(int argc, const struct doveadm_cmd_param *params, + const char *name, ARRAY_TYPE(const_string) **value_r); +bool doveadm_cmd_param_istream(int argc, const struct doveadm_cmd_param *params, + const char *name, struct istream **value_r); void doveadm_cmd_params_clean(ARRAY_TYPE(doveadm_cmd_param_arr_t) *pargv);