[PATCH v2 0/6] improving "git stash list -p"
- From
Jeff King <peff@peff.net>
- Date
- Jul 29, 2014, 17:53 UTC
- Message-ID
- <20140729175300.GA21536@peff.net>
- In-Reply-To
- <20140729115334.GA8976@peff.net>
I spent a few more minutes cleaning up the rough edges here, and this is what I ended up with. This version uses run_diff to implement the actual single-parent diff, so that it matches a "real" diff exactly. It also introduces a few new patches to let you really specify an empty --pretty user-format.
I do think this is an improvement over the current state. I'd also be OK with just specifying "--first-parent" to "git stash list" by default, so that "git stash list -p" does something sensible. I don't like that it doesn't let you peek at the index, but nobody has complained about that in "git stash show", so perhaps nobody cares. All things being equal, I prefer this solution, though.
[1/6]: revision: drop useless string offset when parsing "--pretty" [2/6]: pretty: treat "--format=" as an empty userformat [3/6]: pretty: make empty userformats truly empty [4/6]: add --simplify-combined-diff option [5/6]: stash: default listing to "--cc --simplify-combined-diff" [6/6]: stash: show combined diff with "stash show"
-Peff