Advertising

Perhaps I was vague. "find the commit object and die if that object
is not properly formed" is a good thing. I was referring to the
fact that you
- first had to do get-sha1 yourself to turn the extended sha1
you got from the user into a binary object name, and die with
your own error message if the user input was malformed.
- and then had to call lookup-commit-or-die to do the checking
and let it die.
It would have been simpler for *this* particular codepath if we had
another helper function you can use like so:
commit = lookup_commit_with_extended_sha1_or_die(old_ref);
which did the two-call sequence you handrolled above, and I was
wondering if other existing callers to lookup-commit-or-die wished
the same thing.
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html