ocfs2: one more warning fix in ocfs2_file_aio_write(), v2
authorColy Li <coly.li@suse.de>
Thu, 25 Mar 2010 21:15:12 +0000 (05:15 +0800)
committerJoel Becker <joel.becker@oracle.com>
Tue, 30 Mar 2010 19:52:13 +0000 (12:52 -0700)
This patch fixes another compiling warning in ocfs2_file_aio_write() like this,
    fs/ocfs2/file.c: In function ‘ocfs2_file_aio_write’:
    fs/ocfs2/file.c:2026: warning: suggest parentheses around ‘&&’ within ‘||’

As Joel suggested, '!ret' is unary, this version removes the wrap from '!ret'.

Signed-off-by: Coly Li <coly.li@suse.de>
Signed-off-by: Joel Becker <joel.becker@oracle.com>
fs/ocfs2/file.c

index 17947dc8341e95d187ed13fa69f1a4bdaf5803dc..2b4235c5831dc755deda87c9b0afac0e6604b821 100644 (file)
@@ -2021,9 +2021,9 @@ out_dio:
                if (ret < 0)
                        written = ret;
 
-               if (!ret && (old_size != i_size_read(inode) ||
-                   old_clusters != OCFS2_I(inode)->ip_clusters ||
-                   has_refcount)) {
+               if (!ret && ((old_size != i_size_read(inode)) ||
+                            (old_clusters != OCFS2_I(inode)->ip_clusters) ||
+                            has_refcount)) {
                        ret = jbd2_journal_force_commit(osb->journal->j_journal);
                        if (ret < 0)
                                written = ret;