diff options
| author | Jeroen Hofstee <jeroen@myspectrum.nl> | 2014-06-11 00:28:47 +0200 |
|---|---|---|
| committer | Tom Rini <trini@ti.com> | 2014-06-11 16:27:06 -0400 |
| commit | 930e4254ec29ea5133c0772d4a3804ba1c59b275 (patch) | |
| tree | 912a465f68e4cad5441f00b2d0494225316358ae | |
| parent | 867abdac5effde660ac1ca9db8c43994edc01c09 (diff) | |
| download | u-boot-imx-930e4254ec29ea5133c0772d4a3804ba1c59b275.zip u-boot-imx-930e4254ec29ea5133c0772d4a3804ba1c59b275.tar.gz u-boot-imx-930e4254ec29ea5133c0772d4a3804ba1c59b275.tar.bz2 | |
common/cli_hush.c: remove unnecessary double braces
Clang interpretes an if condition like "if ((a = b) == NULL)
as it tries to assign a value in a statement. Hence if you do
"if ((something)) it warns you that you might be confused.
Hence drop the double braces for plane if statements.
Simon Glass <sjg@chromium.org>
Signed-off-by: Jeroen Hofstee <jeroen@myspectrum.nl>
| -rw-r--r-- | common/cli_hush.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/common/cli_hush.c b/common/cli_hush.c index e0c436f..38da5a0 100644 --- a/common/cli_hush.c +++ b/common/cli_hush.c @@ -1840,7 +1840,7 @@ static int run_list_real(struct pipe *pi) if (rmode == RES_DO) { if (!flag_rep) continue; } - if ((rmode == RES_DONE)) { + if (rmode == RES_DONE) { if (flag_rep) { flag_restore = 1; } else { @@ -3569,7 +3569,7 @@ static char **make_list_in(char **inp, char *name) p3 = insert_var_value(inp[i]); p1 = p3; while (*p1) { - if ((*p1 == ' ')) { + if (*p1 == ' ') { p1++; continue; } |