qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH 1/3] util: add a space before an open parenthesi


From: Thomas Huth
Subject: Re: [Qemu-devel] [PATCH 1/3] util: add a space before an open parenthesis to fix checkpatch errors
Date: Fri, 29 Mar 2019 16:44:30 +0100
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.5.0

On 29/03/2019 16.08, Jules Irenge wrote:
> Add a space before open parenthesis to fix errors issued by checkpatch.pl
> "ERROR: space required before the open parenthesis"
>  within "util/readline.c" file.
> 
> Signed-off-by: Jules Irenge <address@hidden>
> ---
>  util/readline.c | 26 +++++++++++++-------------
>  1 file changed, 13 insertions(+), 13 deletions(-)
> 
> diff --git a/util/readline.c b/util/readline.c
> index ec91ee0fea..a97b600428 100644
> --- a/util/readline.c
> +++ b/util/readline.c
> @@ -48,13 +48,13 @@ static void readline_update(ReadLineState *rs)
>  
>      if (rs->cmd_buf_size != rs->last_cmd_buf_size ||
>          memcmp(rs->cmd_buf, rs->last_cmd_buf, rs->cmd_buf_size) != 0) {
> -        for(i = 0; i < rs->last_cmd_buf_index; i++) {
> +        for (i = 0; i < rs->last_cmd_buf_index; i++) {
>              rs->printf_func(rs->opaque, "\033[D");
>          }
>          rs->cmd_buf[rs->cmd_buf_size] = '\0';
>          if (rs->read_password) {
>              len = strlen(rs->cmd_buf);
> -            for(i = 0; i < len; i++)
> +            for (i = 0; i < len; i++)
>                  rs->printf_func(rs->opaque, "*");

While you're at it, could you please also add curly braces for this
for-loop? ... that's mandated by the QEMU coding conventions, too.

And since there is no explicit maintainer for the util folder and this
is a straight forward clean-up, please also put address@hidden
on CC: when you send the next iteration - that should help to get these
patches merged more easily.

 Thanks,
  Thomas



reply via email to

[Prev in Thread] Current Thread [Next in Thread]