* Is diff.indentHeuristic still experimental?
@ 2019-08-15 4:38 Hong Xu
2019-08-15 9:12 ` [PATCH] diff: 'diff.indentHeuristic' is no longer experimental SZEDER Gábor
0 siblings, 1 reply; 4+ messages in thread
From: Hong Xu @ 2019-08-15 4:38 UTC (permalink / raw)
To: git
Hi all,
I saw that diff.indentHeuristic was enabled by default at
33de7163879ca83be7d6e1583c125f84e1e7c329
<https://git.kernel.org/pub/scm/git/git.git/commit/?id=33de7163879ca83be7d6e1583c125f84e1e7c329>
, but the doc still says it is experimental
<https://git-scm.com/docs/git-config#Documentation/git-config.txt-diffindentHeuristic>.
Am I missing anything?
Best,
Hong
Hong
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH] diff: 'diff.indentHeuristic' is no longer experimental
2019-08-15 4:38 Is diff.indentHeuristic still experimental? Hong Xu
@ 2019-08-15 9:12 ` SZEDER Gábor
2019-08-15 13:56 ` Jeff King
0 siblings, 1 reply; 4+ messages in thread
From: SZEDER Gábor @ 2019-08-15 9:12 UTC (permalink / raw)
To: Junio C Hamano; +Cc: Hong Xu, git, SZEDER Gábor
The indent heuristic started out as experimental, but it's now our
default diff heuristic since 33de716387 (diff: enable indent heuristic
by default, 2017-05-08). Alas, that commit didn't update the
documentation, and the description of the 'diff.indentHeuristic'
configuration variable still implies that it's experimental and not
the default.
Update the description of 'diff.indentHeuristic' to make it clear that
it's the default diff heuristic.
The description of the related '--indent-heuristic' option has already
been updated in bab76141da (diff: --indent-heuristic is no
longer experimental, 2017-10-29).
Signed-off-by: SZEDER Gábor <szeder.dev@gmail.com>
---
Documentation/config/diff.txt | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/Documentation/config/diff.txt b/Documentation/config/diff.txt
index 5afb5a2cbc..ff09f1cf73 100644
--- a/Documentation/config/diff.txt
+++ b/Documentation/config/diff.txt
@@ -189,7 +189,7 @@ diff.guitool::
include::../mergetools-diff.txt[]
diff.indentHeuristic::
- Set this option to `true` to enable experimental heuristics
+ Set this option to `false` to disable the default heuristics
that shift diff hunk boundaries to make patches easier to read.
diff.algorithm::
--
2.23.0.rc2.366.g16015fd804
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] diff: 'diff.indentHeuristic' is no longer experimental
2019-08-15 9:12 ` [PATCH] diff: 'diff.indentHeuristic' is no longer experimental SZEDER Gábor
@ 2019-08-15 13:56 ` Jeff King
2019-08-15 16:58 ` Junio C Hamano
0 siblings, 1 reply; 4+ messages in thread
From: Jeff King @ 2019-08-15 13:56 UTC (permalink / raw)
To: SZEDER Gábor; +Cc: Junio C Hamano, Hong Xu, git
On Thu, Aug 15, 2019 at 11:12:45AM +0200, SZEDER Gábor wrote:
> The indent heuristic started out as experimental, but it's now our
> default diff heuristic since 33de716387 (diff: enable indent heuristic
> by default, 2017-05-08). Alas, that commit didn't update the
> documentation, and the description of the 'diff.indentHeuristic'
> configuration variable still implies that it's experimental and not
> the default.
>
> Update the description of 'diff.indentHeuristic' to make it clear that
> it's the default diff heuristic.
>
> The description of the related '--indent-heuristic' option has already
> been updated in bab76141da (diff: --indent-heuristic is no
> longer experimental, 2017-10-29).
Yeah, this makes perfect sense.
I wondered briefly if we needed to distinguish between plumbing and
porcelain here, but the config option (and its default) are the same for
both.
-Peff
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] diff: 'diff.indentHeuristic' is no longer experimental
2019-08-15 13:56 ` Jeff King
@ 2019-08-15 16:58 ` Junio C Hamano
0 siblings, 0 replies; 4+ messages in thread
From: Junio C Hamano @ 2019-08-15 16:58 UTC (permalink / raw)
To: Jeff King; +Cc: SZEDER Gábor, Hong Xu, git
Jeff King <peff@peff.net> writes:
> On Thu, Aug 15, 2019 at 11:12:45AM +0200, SZEDER Gábor wrote:
>
>> The indent heuristic started out as experimental, but it's now our
>> default diff heuristic since 33de716387 (diff: enable indent heuristic
>> by default, 2017-05-08). Alas, that commit didn't update the
>> documentation, and the description of the 'diff.indentHeuristic'
>> configuration variable still implies that it's experimental and not
>> the default.
>>
>> Update the description of 'diff.indentHeuristic' to make it clear that
>> it's the default diff heuristic.
>>
>> The description of the related '--indent-heuristic' option has already
>> been updated in bab76141da (diff: --indent-heuristic is no
>> longer experimental, 2017-10-29).
>
> Yeah, this makes perfect sense.
OK. Thanks, all.
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2019-08-15 16:58 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-08-15 4:38 Is diff.indentHeuristic still experimental? Hong Xu
2019-08-15 9:12 ` [PATCH] diff: 'diff.indentHeuristic' is no longer experimental SZEDER Gábor
2019-08-15 13:56 ` Jeff King
2019-08-15 16:58 ` Junio C Hamano
Code repositories for project(s) associated with this public inbox
https://80x24.org/mirrors/git.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).