Re: [PATCH v2] sequencer: beautify subject of reverts of reverts

Oswald Buddenhagen <oswald.buddenhagen@gmx.de> writes:

True, but instead of dismissing it (or ^n) as "too nerdy", let's compare it with what we are trying to achieve and see why we feel it is not desirable. I think we are trying to find a good balance between aesthetics and usefulness. The former should take lower precedence, as it would be more subjective between the two.

The usefulness of the message comes from its information content. What do we want to read out of these messages? I think we want a title that immediately lets us know three things:

 (1) What the original patch was about.  
 (2) What the final state is.
 (3) How involved was the road to get to the final state has been.

As to (1), we are not proposing to lose what comes "Revert", so this information is not lost under any proposal we have seen so far in the discussion.

As to (2), with the current "Revert" -> "Revert Revert" -> "Revert Revert Revert" -> ..., you have to count, which is cumbersome and does not give you an immediate access to that information. With "Revert^n", you'd see if n is even or odd to determine, which is much better than the status quo, but it takes practice to interpret. With "Revert" -> "Reapply" -> "Revert Reapply" -> "Reapply Reapply" -> ..., the first word would give you the final state immediately.

We want to know (3), because between a change whose revert was reverted and a change that hasn't been involved in any revert, there may be no difference in the end result, the former is likely to be trickier and merits more careful inspection than the latter. With "Revert^n", we read how large the number n is to find the information. With the current "the Revert repeated number of times" or your "a pair of frontmost Reverts become one Reapply", the length of the Revert/Reapply prefix conveys this information, but this is associated with the cost of pushing the original title further to the right and hard to read/find. Note that, while the number of times revert-reapply sequence took place is a useful piece of information, the exact number may not be all that important.

And from the above discussion, I wonder if the following would be a good place to stop:

 - The first revert is as before:         Revert "original title"
 - A revert of a revert becomes:          Reapply "original title"
 - A revert of a reapply becomes:         Revert Reapply "original title"
 - A revert of "Revert Reapply" becomes:  Reapply Reapply "original title"
 - A revert of "Reapply Reapply" becomes: Revert Reapply "original title"

In other words, we accept the fact that wedo not need exact number of times reversions were done, and use that to simplify the output to make sure we will not spend more than two words in the front of the title. That would help to keep the original title visible, while still allowing us to distinguish the ones that was reverted up to four times (and "Revert Reapply" and "Reapply Reapply" only tell us "final state is to (discard|accept) the original but it took us _many_ times", without saying exactly how many).