[PATCH 2/6] staging: erofs: formatting fix to NULL comparison
Gao Xiang
hsiangkao at aol.com
Fri Aug 31 02:09:34 AEST 2018
Hi Pavel,
On 2018/8/30 23:13, Pavel Zemlyanoy wrote:
> This patch does not change the logic, it only
> corrects the formatting and checkpatch checks by
> to NULL comparison.
>
> The patch fixes 5 checks of type:
> "Comparison to NULL could be written".
>
> Signed-off-by: Pavel Zemlyanoy <zemlyanoy at ispras.ru>
Sorry about the late reply. I am on vacation now.
Personally, I use "== NULL" or "!= NULL" on purpose, since it is more
emphasized than the checkpatch.pl suggested way, and I tend to use the nullptr explicitly.
BTW, It seems other filesystems still use "== NULL" or "!= NULL" explicitly, too, eg:
xfs, ext4, ext2, ocfs2, etc... You could 'grep' in the fs directory...
Other commits look good for me at glance.
Thanks,
Gao Xiang
More information about the Linux-erofs
mailing list