22 Feb
2016
22 Feb
'16
11:15 a.m.
On 22.02.2016 11:29, Martin Kletzander wrote:
Checking whether x > 0 before looping over [0..x] items doesn't make sense and multi-line body must have curly brackets around it.
Best viewed with '-w'.
Signed-off-by: Martin Kletzander <mkletzan@redhat.com> --- src/conf/domain_conf.c | 24 +++++++++++++------- src/qemu/qemu_capabilities.c | 30 ++++++++++++------------ src/storage/storage_backend_rbd.c | 48 +++++++++++++++++++-------------------- src/vbox/vbox_common.c | 14 ++++-------- 4 files changed, 58 insertions(+), 58 deletions(-)
ACK I wonder if we can use Coccinelle to find other occurrences of this pattern. Michal