Markus Heidelberg <markus.heidelberg@xxxxxx> writes: > Is rollback_lock_file(&index_lock) necessary? It isn't used in > "git commit --dry-run" when commit_style is COMMIT_AS_IS. That is because AS_IS commit does not even lock anything for writing, as AS_IS means just that: "git commit" does not touch the index but just writes tree out of the index. Upon program exit (unless you get an uncontrolled crash), the lockfile API arranges atexit(3) to roll back the lockfiles, so it probably may not make much of a difference if you omitted rollback_lock_file(&index_lock) yourself, but it is a good idea to clean up the mess you made after you are done, especially if the mess is not something the operating system will clean up for us (e.g. open file descriptors, malloc'ed region of memory etc.) To make sure that the failure case is covered, you may also want to add a test case where you run "chmod a-w $GIT_DIR" and then run status (but that test needs to be conditional on POSIXPERM). Thanks. -- 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