summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJeroen Hofstee <jeroen@myspectrum.nl>2014-06-11 00:28:47 +0200
committerTom Rini <trini@ti.com>2014-06-11 16:27:06 -0400
commit930e4254ec29ea5133c0772d4a3804ba1c59b275 (patch)
tree912a465f68e4cad5441f00b2d0494225316358ae
parent867abdac5effde660ac1ca9db8c43994edc01c09 (diff)
downloadu-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.c4
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;
}