diff options
| author | Nicholas Marriott <nicm@openbsd.org> | 2009-11-13 19:53:28 +0000 |
|---|---|---|
| committer | Nicholas Marriott <nicm@openbsd.org> | 2009-11-13 19:53:28 +0000 |
| commit | 76ef8770cd983e7ece581fbcfb3a7ad5e115aae5 (patch) | |
| tree | 0b0d7847f179c9963b58488f8c1724e64176266e /cmd-set-option.c | |
| parent | dafa0f022c573513f949c7c66ae83d063fa9a2ab (diff) | |
Get rid of the ugly CMD_CHFLAG macro and use a const string (eg "dDU") in the
command entry structs and a couple of functions to check/set the flags.
Diffstat (limited to 'cmd-set-option.c')
| -rw-r--r-- | cmd-set-option.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/cmd-set-option.c b/cmd-set-option.c index 536b623b..2e00c8d7 100644 --- a/cmd-set-option.c +++ b/cmd-set-option.c @@ -32,7 +32,7 @@ int cmd_set_option_exec(struct cmd *, struct cmd_ctx *); const struct cmd_entry cmd_set_option_entry = { "set-option", "set", "[-agu] " CMD_TARGET_SESSION_USAGE " option [value]", - CMD_ARG12, CMD_CHFLAG('a')|CMD_CHFLAG('g')|CMD_CHFLAG('u'), + CMD_ARG12, "agu", NULL, cmd_target_parse, cmd_set_option_exec, @@ -113,7 +113,7 @@ cmd_set_option_exec(struct cmd *self, struct cmd_ctx *ctx) u_int i; int try_again; - if (data->chflags & CMD_CHFLAG('g')) + if (cmd_check_flag(data->chflags, 'g')) oo = &global_s_options; else { if ((s = cmd_find_session(ctx, data->target)) == NULL) @@ -145,8 +145,8 @@ cmd_set_option_exec(struct cmd *self, struct cmd_ctx *ctx) return (-1); } - if (data->chflags & CMD_CHFLAG('u')) { - if (data->chflags & CMD_CHFLAG('g')) { + if (cmd_check_flag(data->chflags, 'u')) { + if (cmd_check_flag(data->chflags, 'g')) { ctx->error(ctx, "can't unset global option: %s", entry->name); return (-1); @@ -163,7 +163,7 @@ cmd_set_option_exec(struct cmd *self, struct cmd_ctx *ctx) switch (entry->type) { case SET_OPTION_STRING: set_option_string(ctx, oo, entry, - data->arg2, data->chflags & CMD_CHFLAG('a')); + data->arg2, cmd_check_flag(data->chflags, 'a')); break; case SET_OPTION_NUMBER: set_option_number(ctx, oo, entry, data->arg2); |