linux-f2fs-devel.lists.sourceforge.net archive mirror
 help / color / mirror / Atom feed
* linux-4.6-rc2/fs/f2fs/checkpoint.c:214]: (style) Redundant condition
@ 2016-04-06 18:19 David Binderman
  0 siblings, 0 replies; only message in thread
From: David Binderman @ 2016-04-06 18:19 UTC (permalink / raw)
  To: jaegeuk, cm224.lee, chao2.yu, linux-f2fs-devel

Hello there,

linux-4.6-rc2/fs/f2fs/checkpoint.c:214]: (style) Redundant condition: page. '!page || (page && !PageUptodate(page))' is equivalent to '!page || !PageUptodate(page)'

Source code is

    if (!page || (page && !PageUptodate(page)))

Also,

linux-4.6-rc2/fs/f2fs/data.c:1489]: (style) Redundant condition: !err. 'err || (!err && dn.data_blkaddr==NULL_ADDR)' is equivalent to 'err || dn.data_blkaddr==NULL_ADDR'

Source code is

   if (err || (!err && dn.data_blkaddr == NULL_ADDR)) {



Regards

David Binderman
 		 	   		  
------------------------------------------------------------------------------

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2016-04-06 18:19 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-04-06 18:19 linux-4.6-rc2/fs/f2fs/checkpoint.c:214]: (style) Redundant condition David Binderman

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).