Messages in this thread | | | Subject | Re: [PATCH 2/6] staging: erofs: formatting fix to NULL comparison | From | Gao Xiang <> | Date | Mon, 3 Sep 2018 12:23:17 +0800 |
| |
Hi,
On 2018/9/3 9:59, Chao Yu wrote: > On 2018/8/31 17:41, Dan Carpenter wrote: >> On Fri, Aug 31, 2018 at 11:29:03AM +0800, Chao Yu wrote: >>> >>> Hi Xiang, >>> >>> I'm not against this change which follows checkpatch's rule, since I think this >>> can help to unify coding style in different modules of Linux. Maybe cleanup in >>> other filesystem is needed as well. >>> >> >> That code is old, and those filesystems are not in staging so we're not >> going to change them. > > Yup, anyway, it can be decided by their maintainer if they want the cleanup. > >> >> Let's just apply the patch and not spend any time thinking about it. >> Part of the point of style guidelines is so that we don't have to >> repeat all these discussions over and over... >> >> Btw, I have a rename_rev.pl patch for reviewing these. I've attached >> it. rename_rev.pl -r NULL. I've seen some people screw up the >> conversion so having an automated review is nice. > > Cool, thanks very much for providing this tools, it actually can help to save > some time during review. :) >
I am traveling in Tibet. Sorry for the late response.
OK, that is fine. I will follow the new kernel coding style strictly for the upcoming code.
Reviewed-by: Gao Xiang <gaoxiang25@huawei.com>
Thanks, Gao Xiang
>> >> regards, >> dan carpenter >> >
| |