Brandon Casey <casey@xxxxxxxxxxxxxxx> writes: > Mainly, I prefer to not modify the data structures when a failure occurs. Ok. Is the rest of your patch that fixes callers Ok with that semantics? If so, I'd agree that is probably cleaner. I'll scrap the one we are discussing, resurrecting only the api documentation part, and replace it with the lockfile.c changes from your patch, along with the fixes to callers. - 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