> + if (status == -3) {
> + /* something else exists */
> + error(msg, path, ": perhaps a D/F conflict?");
It would be more helpful it you spelled it out like:
Directory/File conflict?
> + return -1;
> + }
> + die(msg, path, "");
> + }
> +
> + /* Successful unlink is good.. */
> + if (!unlink(path))
> + return 0;
> + /* .. and so is no existing file */
> + if (errno == ENOENT)
> + return 0;
> + /* .. but not some other error (who really cares what?) */
> + return error(msg, path, ": perhaps a D/F conflict?");
ditto.
I know this was how it looked previously too but thats just a lame excuse.
Sam
-
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html