Re: [PATCH 1/3] for-each-ref: introduce %C(...) for color

2 messages, 2 authors, 2016-06-15 · open the first message on its own page

Re: [PATCH 1/3] for-each-ref: introduce %C(...) for color

From: Junio C Hamano <hidden>
Date: 2016-06-15 22:59:10

Ramkumar Ramachandra [off-list ref] writes:
Junio C Hamano wrote:
quoted
This patch is about for-each-ref and your series does not seem to
aim to unify it in any way with pretty-formats, so I would have
expected an enhancement in line with the former, not the latter.
While I might never attempt a unification again, there's no harm in
getting the formats to resemble each other in part.
Yes, but...
it's likely that
users of f-e-r format will be familiar with pretty-formats.
... users of for-each-ref format will be _more_ familiar with
formats used by for-each-ref, and it would make a lot more sense to
keep the syntactic resemblance between existing features to show
magic things in for-each-ref and the new feature to show color
(which is merely one new "magic" to the vocabulary in the context of
for-each-ref), no?

Re: [PATCH 1/3] for-each-ref: introduce %C(...) for color

From: Ramkumar Ramachandra <hidden>
Date: 2016-06-15 22:59:11

Junio C Hamano wrote:
... users of for-each-ref format will be _more_ familiar with
formats used by for-each-ref, and it would make a lot more sense to
keep the syntactic resemblance between existing features to show
magic things in for-each-ref and the new feature to show color
(which is merely one new "magic" to the vocabulary in the context of
for-each-ref), no?
Okay, so what do you suggest in place of %C(...)?
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help