On Mon, Nov 21, 2022 at 07:01:44PM +0100, Tobias Heider wrote:

> +             if (verbose)

This block could use braces.

> +                     fprintf(stderr, "%s %s to %s\n",
> +                         (nowrite ? "would copy" : "copying"),
> +                         src, dst);
> +             if (!nowrite)

Here you forgot them but got lucky that it still works out.

Missing                       {

> +                     rslt = filecopy(src, dst);
> +                     if (rslt == -1)
> +                             goto cleanup;

Missing         }

> +     }
> +     rslt = 0;
> +
> + cleanup:
> +     free(src);
> +     return rslt;
>  }
>  
>  /*
> 

Reply via email to