Message ID | 1577241959-138695-1-git-send-email-zhengbin13@huawei.com |
---|---|
State | Accepted |
Headers | show |
Series | ext4: use true,false for bool variable | expand |
On Wed 25-12-19 10:45:59, zhengbin wrote: > Fixes coccicheck warning: > > fs/ext4/extents.c:5271:6-12: WARNING: Assignment of 0/1 to bool variable > fs/ext4/extents.c:5287:4-10: WARNING: Assignment of 0/1 to bool variable > > Reported-by: Hulk Robot <hulkci@huawei.com> > Signed-off-by: zhengbin <zhengbin13@huawei.com> Looks good to me. You can add: Reviewed-by: Jan Kara <jack@suse.cz> Honza > --- > fs/ext4/extents.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/fs/ext4/extents.c b/fs/ext4/extents.c > index 0e8708b..d8611be 100644 > --- a/fs/ext4/extents.c > +++ b/fs/ext4/extents.c > @@ -5268,7 +5268,7 @@ ext4_ext_shift_path_extents(struct ext4_ext_path *path, ext4_lblk_t shift, > { > int depth, err = 0; > struct ext4_extent *ex_start, *ex_last; > - bool update = 0; > + bool update = false; > depth = path->p_depth; > > while (depth >= 0) { > @@ -5284,7 +5284,7 @@ ext4_ext_shift_path_extents(struct ext4_ext_path *path, ext4_lblk_t shift, > goto out; > > if (ex_start == EXT_FIRST_EXTENT(path[depth].p_hdr)) > - update = 1; > + update = true; > > while (ex_start <= ex_last) { > if (SHIFT == SHIFT_LEFT) { > -- > 2.7.4 >
On Thu, Jan 09, 2020 at 11:22:54AM +0100, Jan Kara wrote: > On Wed 25-12-19 10:45:59, zhengbin wrote: > > Fixes coccicheck warning: > > > > fs/ext4/extents.c:5271:6-12: WARNING: Assignment of 0/1 to bool variable > > fs/ext4/extents.c:5287:4-10: WARNING: Assignment of 0/1 to bool variable > > > > Reported-by: Hulk Robot <hulkci@huawei.com> > > Signed-off-by: zhengbin <zhengbin13@huawei.com> > > Looks good to me. You can add: > > Reviewed-by: Jan Kara <jack@suse.cz> Applied, thanks. - Ted
diff --git a/fs/ext4/extents.c b/fs/ext4/extents.c index 0e8708b..d8611be 100644 --- a/fs/ext4/extents.c +++ b/fs/ext4/extents.c @@ -5268,7 +5268,7 @@ ext4_ext_shift_path_extents(struct ext4_ext_path *path, ext4_lblk_t shift, { int depth, err = 0; struct ext4_extent *ex_start, *ex_last; - bool update = 0; + bool update = false; depth = path->p_depth; while (depth >= 0) { @@ -5284,7 +5284,7 @@ ext4_ext_shift_path_extents(struct ext4_ext_path *path, ext4_lblk_t shift, goto out; if (ex_start == EXT_FIRST_EXTENT(path[depth].p_hdr)) - update = 1; + update = true; while (ex_start <= ex_last) { if (SHIFT == SHIFT_LEFT) {
Fixes coccicheck warning: fs/ext4/extents.c:5271:6-12: WARNING: Assignment of 0/1 to bool variable fs/ext4/extents.c:5287:4-10: WARNING: Assignment of 0/1 to bool variable Reported-by: Hulk Robot <hulkci@huawei.com> Signed-off-by: zhengbin <zhengbin13@huawei.com> --- fs/ext4/extents.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) -- 2.7.4