Skip to contentSkip to conversation

ocfs2: one more warning fix in ocfs2_file_aio_write(), v2

PATCH New Coly Li 2 participants 2 messages Mar 25, 2010 – Apr 1, 2010 OCFS2

lore
Share & tools
lore

Coly Li

author
#0 lore raw

PATCH 1/1 ocfs2: one more warning fix in ocfs2_file_aio_write(), v2

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'.

Cc Joel Becker <[email protected]>
---
 fs/ocfs2/file.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)
fs/ocfs2/file.c +2 -2
index 17947dc..c4638fd 100644
@@ -2021,8 +2021,8 @@ out_dio:
 		if (ret < 0)
 			written = ret;
 
-		if (!ret && (old_size != i_size_read(inode) ||
-		    old_clusters != OCFS2_I(inode)->ip_clusters ||
+		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)
Signature · 2 lines
Coly Li
SuSE Labs
#1 lore raw

PATCH 1/1 ocfs2: one more warning fix in ocfs2_file_aio_write(), v2

Quoted 5 lines · Coly Li

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'.

This patch is now in the fixes branch of ocfs2.git.

Joel

Signature · 8 lines
"You don't make the poor richer by making the rich poorer."
	- Sir Winston Churchill

Joel Becker
Principal Software Developer
Oracle
E-mail: joel.becker at oracle.com
Phone: (650) 506-8127