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; > } > > /* >