2022-07-22 19:38:26

by Xin Gao

[permalink] [raw]
Subject: [PATCH] xfs: Fix comment typo

The double `for' is duplicated in line 3788, remove one.

Signed-off-by: Xin Gao <[email protected]>
---
fs/xfs/libxfs/xfs_bmap.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/fs/xfs/libxfs/xfs_bmap.c b/fs/xfs/libxfs/xfs_bmap.c
index 6833110d1bd4..6f6d87ba29a3 100644
--- a/fs/xfs/libxfs/xfs_bmap.c
+++ b/fs/xfs/libxfs/xfs_bmap.c
@@ -3785,7 +3785,7 @@ xfs_bmapi_trim_map(
mval->br_startblock = got->br_startblock +
(*bno - got->br_startoff);
/*
- * Return the minimum of what we got and what we asked for for
+ * Return the minimum of what we got and what we asked for
* the length. We can use the len variable here because it is
* modified below and we could have been there before coming
* here if the first part of the allocation didn't overlap what
--
2.30.2


2022-07-22 20:38:15

by Eric Sandeen

[permalink] [raw]
Subject: Re: [PATCH] xfs: Fix comment typo

On 7/22/22 2:36 PM, Xin Gao wrote:
> The double `for' is duplicated in line 3788, remove one.
>
> Signed-off-by: Xin Gao <[email protected]>
> ---
> fs/xfs/libxfs/xfs_bmap.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/fs/xfs/libxfs/xfs_bmap.c b/fs/xfs/libxfs/xfs_bmap.c
> index 6833110d1bd4..6f6d87ba29a3 100644
> --- a/fs/xfs/libxfs/xfs_bmap.c
> +++ b/fs/xfs/libxfs/xfs_bmap.c
> @@ -3785,7 +3785,7 @@ xfs_bmapi_trim_map(
> mval->br_startblock = got->br_startblock +
> (*bno - got->br_startoff);
> /*
> - * Return the minimum of what we got and what we asked for for
> + * Return the minimum of what we got and what we asked for
> * the length. We can use the len variable here because it is
> * modified below and we could have been there before coming
> * here if the first part of the allocation didn't overlap what

NAK.

The comment is correct.