mirror of
https://xff.cz/git/u-boot/
synced 2025-09-04 18:22:02 +02:00
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>
This commit is contained in:
@@ -1840,7 +1840,7 @@ static int run_list_real(struct pipe *pi)
|
|||||||
if (rmode == RES_DO) {
|
if (rmode == RES_DO) {
|
||||||
if (!flag_rep) continue;
|
if (!flag_rep) continue;
|
||||||
}
|
}
|
||||||
if ((rmode == RES_DONE)) {
|
if (rmode == RES_DONE) {
|
||||||
if (flag_rep) {
|
if (flag_rep) {
|
||||||
flag_restore = 1;
|
flag_restore = 1;
|
||||||
} else {
|
} else {
|
||||||
@@ -3569,7 +3569,7 @@ static char **make_list_in(char **inp, char *name)
|
|||||||
p3 = insert_var_value(inp[i]);
|
p3 = insert_var_value(inp[i]);
|
||||||
p1 = p3;
|
p1 = p3;
|
||||||
while (*p1) {
|
while (*p1) {
|
||||||
if ((*p1 == ' ')) {
|
if (*p1 == ' ') {
|
||||||
p1++;
|
p1++;
|
||||||
continue;
|
continue;
|
||||||
}
|
}
|
||||||
|
Reference in New Issue
Block a user