# HG changeset patch
# User Steve Fink <sfink@mozilla.com>
# Date 1454696031 28800
#      Fri Feb 05 10:13:51 2016 -0800
# Node ID 37127b1ed77749cf6b722581616e425d6d371b3e
# Parent  0fffe289b31629201908175adf9cb4fb048cb929
chistedit: show direction of fold and roll with caret

diff --git a/chistedit.py b/chistedit.py
--- a/chistedit.py
+++ b/chistedit.py
@@ -64,6 +64,10 @@
         'r': 'roll',
     }
 KEY_LIST = ['pick', 'edit', 'fold', 'drop', 'mess', 'roll']
+ACTION_LABELS = {
+    'fold': '^fold',
+    'roll': '^roll',
+}
 
 COLOR_HELP, COLOR_SELECTED, COLOR_OK, COLOR_WARN  = 1, 2, 3, 4
 
@@ -79,11 +83,24 @@
         self.conflicts = []
 
     def __str__(self):
-        action = self.action
+        # Some actions ('fold' and 'roll') combine a patch with a previous one.
+        # Add a marker showing which patch they apply to, and also omit the
+        # description for 'roll' (since it will get discarded). Example display:
+        #
+        #  #10 pick   316392:06a16c25c053   add option to skip tests
+        #  #11 ^roll  316393:71313c964cc5
+        #  #12 pick   316394:ab31f3973b0d   include mfbt for mozilla-config.h
+        #  #13 ^fold  316395:14ce5803f4c3   fix warnings
+        #
+        # The carets point to the changeset being folded into ("roll this
+        # changeset into the changeset above").
+        action = ACTION_LABELS.get(self.action, self.action)
         h = self.ctx.hex()[0:12]
         r = self.ctx.rev()
         desc = self.ctx.description().splitlines()[0].strip()
-        return "#{0:<2} {1}   {2}:{3}   {4}".format(
+        if self.action == 'roll':
+            desc = ''
+        return "#{0:<2} {1:<6} {2}:{3}   {4}".format(
                 self.origpos, action, r, h, desc)
 
     def checkconflicts(self, other):