On Sat, Sep 28, 2013 at 5:10 PM, Kirill A. Shutemov <kirill.shutemov@xxxxxxxxxxxxxxx> wrote: > collapse_todo_ids() uses `git rev-parse --short=7' to abbrev commit ids. > Let's drop argument from --short to use default (7) or config value > instead. > > Signed-off-by: Kirill A. Shutemov <kirill.shutemov@xxxxxxxxxxxxxxx> > --- > git-rebase--interactive.sh | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/git-rebase--interactive.sh b/git-rebase--interactive.sh > index 10bf318d0d..3c6bed9a28 100644 > --- a/git-rebase--interactive.sh > +++ b/git-rebase--interactive.sh > @@ -713,7 +713,7 @@ expand_todo_ids() { > } > > collapse_todo_ids() { > - transform_todo_ids --short=7 > + transform_todo_ids --short Thanks, I was going to suggest this exact simplification upon reading v1. Acked-by: Eric Sunshine <sunshine@xxxxxxxxxxxxxx> > } > > # Rearrange the todo list that has both "pick sha1 msg" and > -- > Kirill A. Shutemov -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html