Re: [RFC PATCH] show decorations at the end of the line

On Tue, Feb 21, 2017 at 12:11 PM, Junio C Hamano <gitster@pobox.com> wrote:

So I have been using the original patch for a bit over a week now, and I have to say that I'm not sure it's the right thing to do after all.

Most of the time I much prefer the "decorations at the end" thing, because it just looks better, and the commit log oneliners line up nicely.

But then occasionally I end up liking the old interface better, just because there's long commit lines, and showing the decoration at the end effectively hides it.

So I vacillate between the two formats, and so I'm not sure this patch is worth the change in behavior after all.

In fact, I played around with some formats, and the one I lines the most was actually one that split the line for decorations, but that one was admittedly pretty funky. It gives output like

  b9df16a4c (HEAD -> master)
            pathspec: don't error out on all-exclusionary pathspec patterns
  91a491f05 pathspec magic: add '^' as alias for '!'
  c8e05fd6d ls-remote: add "--diff" option to show only refs that differ
  20769079d (tag: v2.12.0-rc2, origin/master, origin/HEAD)
            Git 2.12-rc2
  076c05393 Hopefully the final batch of mini-topics before the final
  c5b22b819 Merge branch 'jk/tempfile-ferror-fclose-confusion'
  62fef5c56 Merge branch 'dp/submodule-doc-markup-fix'
  1f73ff080 Merge branch 'jk/reset-to-break-a-commit-doc-updated'
  bf5f11918 Merge branch 'jk/reset-to-break-a-commit-doc'
  e048a257b Merge branch 'js/mingw-isatty'

(which looks better with colorization than it looks in the email).

But I'm not even going to send out that patch, because it was such an atrocious hack to line things up.

              Linus