Edmundo Carmona Antoranz <eantoranz@xxxxxxxxx> writes:
> Fair enough. It's ok to explain here, right?
Yes, absolutely. How it works is
- an author comes up with a patch and its explanation and
justificaiton,
- reviewers ask clarifications on what was unclear, point out what
was wrong in the code, etc., and then
- the author responds with "here is what I meant", "yeah, you're
right that the code is wrong--would it be right to write it this
way?", and/or "not really, the code is OK because ...", etc.
All that exchange is to come up with a more polished version that
has correct code that is explained and justified in a clear manner.
The next iteration the author then sends would of course include the
fixes to bugs pointed out during the review, but also would explain
and justify the change better---the explanation in the original did
not convey what the author wanted to tell, but the explanation the
author gave during the review discussion did so better, so the new
iteration can learn from that exchange.
> There's no simple way to stash only
> what you have on index while retaining your working tree state.
Yes, that much can be inferred from what it does ("there is no easy
way to do X, so I am adding X").
The question is why would one want to do X in the first place.
IIUC, the simplest workflow using the new feature may go like this:
$ edit file ;# edit a bit
$ git add file
$ edit file ;# even even more
$ git stash --cached
After all of the above is done, "git stash list" may show that there
is a single stash that records the changes you made to the file
right after you added it, without your further changes (because you
are taking what is in the index). Your working tree has all the
changes you made to file, both before and after "git add", and your
index is clean.
After you got yourself into this state where your index is clean,
your working tree file has all changes, and your stash entry has
only the earlier half of the change, what are you going to do with
that stash entry? If we learn the answer to that question, perhaps
we may find (or we may even have) a better or easier way to achieve
whatever you were planning to do with that stash entry by some other
means---it might not even involve "git stash"---but without knowing
that, we cannot tell if the new feature is a good idea.
Thanks.
