2019-08-27 08:52:59

by Dan Carpenter

[permalink] [raw]
Subject: [PATCH] ext4: tidy up white space in count_rsvd()

This line was indented one tab too far.

Signed-off-by: Dan Carpenter <[email protected]>
---
fs/ext4/extents_status.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/fs/ext4/extents_status.c b/fs/ext4/extents_status.c
index dc28a9642452..f17e3f521a17 100644
--- a/fs/ext4/extents_status.c
+++ b/fs/ext4/extents_status.c
@@ -1089,7 +1089,7 @@ static void count_rsvd(struct inode *inode, ext4_lblk_t lblk, long len,
*/
if ((i + sbi->s_cluster_ratio - 1) <= end) {
nclu = (end - i + 1) >> sbi->s_cluster_bits;
- rc->ndelonly += nclu;
+ rc->ndelonly += nclu;
i += nclu << sbi->s_cluster_bits;
}

--
2.20.1


2019-08-27 20:57:45

by Eric Whitney

[permalink] [raw]
Subject: Re: [PATCH] ext4: tidy up white space in count_rsvd()

* Dan Carpenter <[email protected]>:
> This line was indented one tab too far.
>
> Signed-off-by: Dan Carpenter <[email protected]>

Good catch, thanks. You can add:

Reviewed-by: Eric Whitney <[email protected]>

> ---
> fs/ext4/extents_status.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/fs/ext4/extents_status.c b/fs/ext4/extents_status.c
> index dc28a9642452..f17e3f521a17 100644
> --- a/fs/ext4/extents_status.c
> +++ b/fs/ext4/extents_status.c
> @@ -1089,7 +1089,7 @@ static void count_rsvd(struct inode *inode, ext4_lblk_t lblk, long len,
> */
> if ((i + sbi->s_cluster_ratio - 1) <= end) {
> nclu = (end - i + 1) >> sbi->s_cluster_bits;
> - rc->ndelonly += nclu;
> + rc->ndelonly += nclu;
> i += nclu << sbi->s_cluster_bits;
> }
>
> --
> 2.20.1
>

2019-08-28 15:33:28

by Theodore Ts'o

[permalink] [raw]
Subject: Re: [PATCH] ext4: tidy up white space in count_rsvd()

On Tue, Aug 27, 2019 at 04:54:45PM -0400, Eric Whitney wrote:
> * Dan Carpenter <[email protected]>:
> > This line was indented one tab too far.
> >
> > Signed-off-by: Dan Carpenter <[email protected]>
>
> Good catch, thanks. You can add:
>
> Reviewed-by: Eric Whitney <[email protected]>

Thanks, applied by fixing up the original base patch.

- Ted