On Sun, Sep 11, 2022 at 11:31:37AM +0800, Stephen Zhang wrote:
> xfs_dir2_isleaf is used to see if the directory is a single-leaf
> form directory, as commented right above the function.
>
> Signed-off-by: Shida Zhang <zhangshida@xxxxxxxxxx>
> ---
> fs/xfs/libxfs/xfs_dir2.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/fs/xfs/libxfs/xfs_dir2.c b/fs/xfs/libxfs/xfs_dir2.c
> index 76eedc2756b3..1485f53fecf4 100644
> --- a/fs/xfs/libxfs/xfs_dir2.c
> +++ b/fs/xfs/libxfs/xfs_dir2.c
> @@ -632,7 +632,7 @@ xfs_dir2_isblock(
> int
> xfs_dir2_isleaf(
> struct xfs_da_args *args,
> - int *vp) /* out: 1 is block, 0 is not block */
> + int *vp) /* out: 1 is leaf, 0 is not leaf */
> {
> xfs_fileoff_t last; /* last file offset */
> int rval;
If you are going to touch this code to fix a broken comment, then
please clean it up properly.
The "*vp" parameter should be a "bool *isleaf", in which case the
return value is obvious and the comment can be removed. Then the
logic in the function can be cleaned up to be obvious instead of
relying on easy to mistake conditional logic in assignemnts...
IOWs, don't fix the comments - fix the code to be self documenting
to remove the need for comments...
Cheers,
Dave.
--
Dave Chinner
david@xxxxxxxxxxxxx
