[PATCH 0/2] minor show-index modernizations
- From
Jeff King <peff@peff.net>
- Date
- May 28, 2018, 09:37 UTC
- Message-ID
- <20180528093740.GA6902@sigill.intra.peff.net>
While recommending show-index to somebody today, I noticed that it has a few ancient warts. This series turns it into a builtin and fixes the documentation. I suspect it could do with some more modernization and friendliness, like:
- re-using the existing packfile.c code instead of re-implementing
index-reading (though the packfile code would definitely need some
refactoring to make this work) - it could probably handle arguments as files instead of requiring
stdin redirectionI'll leave those for now, as they're not worth the effort to me at this point. But everybody who _doesn't_ use the command benefits from making it a builtin, so it seems like an easy win.
[1/2]: make show-index a builtin [2/2]: show-index: update documentation for index v2
Documentation/git-show-index.txt | 22 ++++++++++++++++------ Makefile | 2 +- builtin.h | 1 + show-index.c => builtin/show-index.c | 2 +- git.c | 1 + 5 files changed, 20 insertions(+), 8 deletions(-) rename show-index.c => builtin/show-index.c (96%)
-Peff