address-a-benign-compiler-warning

   1 ext4: address a benign compiler warning
   2
   3 From: Patrick Palka <patrick@parcs.ath.cx>
   4
   5 When !defined(CONFIG_EXT4_DEBUG), mb_debug() should be defined as a
   6 no_printk() statement instead of an empty statement in order to suppress
   7 the following compiler warning:
   8
   9 fs/ext4/mballoc.c: In function ‘ext4_mb_cleanup_pa’:
  10 fs/ext4/mballoc.c:2659:47: warning: suggest braces around empty body in an ‘if’ statement [-Wempty-body]
  11    mb_debug(1, "mballoc: %u PAs left\n", count);
  12
  13 Signed-off-by: Patrick Palka <patrick@parcs.ath.cx>
  14 Signed-off-by: "Theodore Ts'o" <tytso@mit.edu>
  15 ---
  16  fs/ext4/mballoc.h | 2 +-
  17  1 file changed, 1 insertion(+), 1 deletion(-)
  18
  19 diff --git a/fs/ext4/mballoc.h b/fs/ext4/mballoc.h
  20 index 08481ee..9347328 100644
  21 --- a/fs/ext4/mballoc.h
  22 +++ b/fs/ext4/mballoc.h
  23 @@ -48,7 +48,7 @@ extern ushort ext4_mballoc_debug;
  24                 }                                                       \
  25         } while (0)
  26  #else
  27 -#define mb_debug(n, fmt, a...)
  28 +#define mb_debug(n, fmt, a...)         no_printk(fmt, ## a)
  29  #endif
  30
  31  #define EXT4_MB_HISTORY_ALLOC          1       /* allocation */
  32 --
  33 1.9.0.rc3
  34
  35