diff options
author | Joe Perches <[email protected]> | 2014-08-06 16:10:29 -0700 |
---|---|---|
committer | Linus Torvalds <[email protected]> | 2014-08-06 18:01:27 -0700 |
commit | 032a4c0f9a77ce565355c6e191553e853ba66f09 (patch) | |
tree | 472cf85c05fdcafe028e5b8140729327afcffeea /tools/perf/scripts/python/Perf-Trace-Util/lib/Perf/Trace/SchedGui.py | |
parent | ebfdc40969f24fc0cdd1349835d36e8ebae05374 (diff) |
checkpatch: warn on unnecessary else after return or break
Using an else following a break or return can unnecessarily indent code
blocks.
ie:
for (i = 0; i < 100; i++) {
int foo = bar();
if (foo < 1)
break;
else
usleep(1);
}
is generally better written as:
for (i = 0; i < 100; i++) {
int foo = bar();
if (foo < 1)
break;
usleep(1);
}
Warn when a bare else statement is preceded by a break or return
indented 1 tab more than the else.
Signed-off-by: Joe Perches <[email protected]>
Signed-off-by: Andrew Morton <[email protected]>
Signed-off-by: Linus Torvalds <[email protected]>
Diffstat (limited to 'tools/perf/scripts/python/Perf-Trace-Util/lib/Perf/Trace/SchedGui.py')
0 files changed, 0 insertions, 0 deletions