[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 2/4] linux-user: Use existing envlist_parse_set/
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PATCH 2/4] linux-user: Use existing envlist_parse_set/envlist_parse_unset interface. |
Date: |
Thu, 27 Jun 2013 18:49:16 +0100 |
On 29 May 2013 14:50, Thomas Schwinge <address@hidden> wrote:
> Signed-off-by: Thomas Schwinge <address@hidden>
> ---
> linux-user/main.c | 18 ++++--------------
> util/envlist.c | 4 ++--
> 2 files changed, 6 insertions(+), 16 deletions(-)
>
> diff --git linux-user/main.c linux-user/main.c
> index b97b8cf..a0ea161 100644
> --- linux-user/main.c
> +++ linux-user/main.c
> @@ -3204,26 +3204,16 @@ static void handle_arg_log_filename(const char *arg)
>
> static void handle_arg_set_env(const char *arg)
> {
> - char *r, *p, *token;
> - r = p = strdup(arg);
> - while ((token = strsep(&p, ",")) != NULL) {
> - if (envlist_setenv(envlist, token) != 0) {
> - usage();
> - }
> + if (envlist_parse_set(envlist, arg) != 0) {
> + usage();
> }
> - free(r);
> }
>
> static void handle_arg_unset_env(const char *arg)
> {
> - char *r, *p, *token;
> - r = p = strdup(arg);
> - while ((token = strsep(&p, ",")) != NULL) {
> - if (envlist_unsetenv(envlist, token) != 0) {
> - usage();
> - }
> + if (envlist_parse_unset(envlist, arg) != 0) {
> + usage();
> }
> - free(r);
> }
This looks OK...
>
> static void handle_arg_argv0(const char *arg)
> diff --git util/envlist.c util/envlist.c
> index cbbf7e5..8027bbf 100644
> --- util/envlist.c
> +++ util/envlist.c
> @@ -55,10 +55,10 @@ envlist_free(envlist_t *envlist)
>
> /*
> * Parses comma separated list of set/modify environment
> - * variable entries and updates given enlist accordingly.
> + * variable entries and updates given envlist accordingly.
> *
> * For example:
> - * envlist_parse(el, "HOME=foo,SHELL=/bin/sh");
> + * envlist_parse_set(el, "HOME=foo,SHELL=/bin/sh");
> *
> * inserts/sets environment variables HOME and SHELL.
> *
...but this bit needs to be a separate patch.
-- PMM
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [Qemu-devel] [PATCH 2/4] linux-user: Use existing envlist_parse_set/envlist_parse_unset interface.,
Peter Maydell <=