Re: [PATCH] xfs: scrub: remove unnecessary braces and fix

[Darrick J. Wong]

On Sat, Jul 19, 2025 at 10:22:09AM +0500, or10n-cli wrote:
> From b8e455b79c84b4e1501ea554327672b6d391d35d Mon Sep 17 00:00:00 2001
> From: or10n-cli <muhammad.ahmed.27@xxxxxxxxxxx>

WTF is "orion-cli" ?

> Date: Sat, 19 Jul 2025 10:10:42 +0500
> Subject: [PATCH] xfs: scrub: remove unnecessary braces and fix indentation
> in
>  findparent.c
> 
> This patch removes unnecessary braces around simple if-else blocks and
> fixes inconsistent indentation in fs/xfs/scrub/findparent.c to comply
> with kernel coding style guidelines.
> 
> All changes are verified using checkpatch.pl with no warnings or errors.
> 
> Signed-off-by: Muhammad Ahmed <muhammad.ahmed.27@xxxxxxxxxxx>
> ---
>  fs/xfs/scrub/findparent.c | 9 +++------
>  1 file changed, 3 insertions(+), 6 deletions(-)
> 
> diff --git a/fs/xfs/scrub/findparent.c b/fs/xfs/scrub/findparent.c
> index 84487072b6dd..9a2f25c7c2e3 100644
> --- a/fs/xfs/scrub/findparent.c
> +++ b/fs/xfs/scrub/findparent.c
> @@ -229,15 +229,12 @@ xrep_findparent_live_update(
>          */
>         if (p->ip->i_ino == sc->ip->i_ino &&
>             xchk_iscan_want_live_update(&pscan->iscan, p->dp->i_ino)) {
> -               if (p->delta > 0) {
> +               if (p->delta > 0)
>                         xrep_findparent_scan_found(pscan, p->dp->i_ino);
> -               } else {
> +               else
>                         xrep_findparent_scan_found(pscan, NULLFSINO);
> -               }
>         }
> -
>         return NOTIFY_DONE;
> -}

DID YOU EVEN COMPILE THIS???

--D

>  /*
>   * Set up a scan to find the parent of a directory.  The provided dirent
> hook
> @@ -386,7 +383,7 @@ xrep_findparent_confirm(
> 
>         /* Reject garbage parent inode numbers and self-referential parents.
> */
>         if (*parent_ino == NULLFSINO)
> -              return 0;
> +               return 0;
>         if (!xfs_verify_dir_ino(sc->mp, *parent_ino) ||
>             *parent_ino == sc->ip->i_ino) {
>                 *parent_ino = NULLFSINO;
> --
> 2.47.2
>