Jeff King <peff@xxxxxxxx> writes: > On Sat, Jan 21, 2017 at 08:08:02PM +0000, Thomas Gummerer wrote: > >> diff --git a/Documentation/git-stash.txt b/Documentation/git-stash.txt >> index 2e9cef06e6..0ad5335a3e 100644 >> --- a/Documentation/git-stash.txt >> +++ b/Documentation/git-stash.txt >> @@ -47,8 +47,9 @@ OPTIONS >> >> save [-p|--patch] [-k|--[no-]keep-index] [-u|--include-untracked] [-a|--all] [-q|--quiet] [<message>]:: >> >> - Save your local modifications to a new 'stash', and run `git reset >> - --hard` to revert them. The <message> part is optional and gives >> + Save your local modifications to a new 'stash', and revert the >> + the changes in the working tree to match the index. >> + The <message> part is optional and gives > > Hrm. "git reset --hard" doesn't just make the working tree match the > index. It also resets the index to HEAD. So either the original or your > new description is wrong. > > I think it's the latter. We really do reset the index unless > --keep-index is specified. Correct. So the patch is a net loss. Perhaps not requiring readers to know "reset --hard" might be an improvement (I personally do not think so), but this loses crucial information from the description. Save your local modifications (both in the working tree and to the index) to a new 'stash', and resets the index to HEAD and makes the working tree match the index (i.e. runs "git reset --hard"). That's one-and-a-half lines longer than the original, though.