]> asedeno.scripts.mit.edu Git - linux.git/commitdiff
nilfs2: remove loops of single statement macros
authorRyusuke Konishi <konishi.ryusuke@lab.ntt.co.jp>
Mon, 23 May 2016 23:23:45 +0000 (16:23 -0700)
committerLinus Torvalds <torvalds@linux-foundation.org>
Tue, 24 May 2016 00:04:14 +0000 (17:04 -0700)
This fixes checkpatch.pl warning "WARNING: Single statement macros
should not use a do {} while (0) loop".

Link: http://lkml.kernel.org/r/1462886671-3521-7-git-send-email-konishi.ryusuke@lab.ntt.co.jp
Signed-off-by: Ryusuke Konishi <konishi.ryusuke@lab.ntt.co.jp>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
fs/nilfs2/the_nilfs.h

index 06d2548d436d093602b808c48f21aa074c6d429c..79369fd6b13bcb8ba73198887f004fbf8536b663 100644 (file)
@@ -222,15 +222,14 @@ THE_NILFS_FNS(SB_DIRTY, sb_dirty)
  * Mount option operations
  */
 #define nilfs_clear_opt(nilfs, opt)  \
-       do { (nilfs)->ns_mount_opt &= ~NILFS_MOUNT_##opt; } while (0)
+       ((nilfs)->ns_mount_opt &= ~NILFS_MOUNT_##opt)
 #define nilfs_set_opt(nilfs, opt)  \
-       do { (nilfs)->ns_mount_opt |= NILFS_MOUNT_##opt; } while (0)
+       ((nilfs)->ns_mount_opt |= NILFS_MOUNT_##opt)
 #define nilfs_test_opt(nilfs, opt) ((nilfs)->ns_mount_opt & NILFS_MOUNT_##opt)
 #define nilfs_write_opt(nilfs, mask, opt)                              \
-       do { (nilfs)->ns_mount_opt =                                    \
+       ((nilfs)->ns_mount_opt =                                        \
                (((nilfs)->ns_mount_opt & ~NILFS_MOUNT_##mask) |        \
-                NILFS_MOUNT_##opt);                                    \
-       } while (0)
+                NILFS_MOUNT_##opt))                                    \
 
 /**
  * struct nilfs_root - nilfs root object