Thread (7 messages) 7 messages, 6 authors, 2017-04-28

Re: [PATCH] diff: prefer indent heuristic over compaction heuristic

From: Jacob Keller <hidden>
Date: 2016-12-23 22:23:52

Possibly related (same subject, not in this thread)

On Fri, Dec 23, 2016 at 2:21 PM, Jeff King [off-list ref] wrote:
On Fri, Dec 23, 2016 at 01:17:03PM -0800, Junio C Hamano wrote:
quoted
Junio C Hamano [off-list ref] writes:
quoted
I guess both you and Michael are in favor of just removing compaction
variant without any renames, so let me prepare a reroll and queue
that instead.  We can flip the default perhaps one release later.
-- >8 --
Subject: [PATCH] diff: retire "compaction" heuristics
Looks good to me from a cursory read.

Thanks.

-Peff
Same. This is more obviously correct since we didn't have to change a
bunch of references to INDENT_HEURISTIC. I agree that the name does
not make sense now, but if our goal is to make it default with a
disable option, I think that we shouldn't worry too much about the
naming.

Thanks,
Jake
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help