* [PATCH] blame: fix typo in documentation @ 2021-06-18 22:11 Andrei Rybak 2021-06-19 5:27 ` Bagas Sanjaya 2021-06-19 11:29 ` [PATCH v2] blame: correct name of config option in docs Andrei Rybak 0 siblings, 2 replies; 16+ messages in thread From: Andrei Rybak @ 2021-06-18 22:11 UTC (permalink / raw) To: git; +Cc: Barret Rhoden As can be seen in files "Documentation/blame-options.txt" and "builtin/blame.c", the name of this configuration variable is "blame.markUnblamableLines". Signed-off-by: Andrei Rybak <rybak.a.v@gmail.com> --- Documentation/config/blame.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/config/blame.txt b/Documentation/config/blame.txt index 9468e8599c..4d047c1790 100644 --- a/Documentation/config/blame.txt +++ b/Documentation/config/blame.txt @@ -27,7 +27,7 @@ blame.ignoreRevsFile:: file names will reset the list of ignored revisions. This option will be handled before the command line option `--ignore-revs-file`. -blame.markUnblamables:: +blame.markUnblamableLines:: Mark lines that were changed by an ignored revision that we could not attribute to another commit with a '*' in the output of linkgit:git-blame[1]. -- 2.32.0 ^ permalink raw reply related [flat|nested] 16+ messages in thread
* Re: [PATCH] blame: fix typo in documentation 2021-06-18 22:11 [PATCH] blame: fix typo in documentation Andrei Rybak @ 2021-06-19 5:27 ` Bagas Sanjaya 2021-06-19 11:12 ` Andrei Rybak 2021-06-19 17:31 ` Felipe Contreras 2021-06-19 11:29 ` [PATCH v2] blame: correct name of config option in docs Andrei Rybak 1 sibling, 2 replies; 16+ messages in thread From: Bagas Sanjaya @ 2021-06-19 5:27 UTC (permalink / raw) To: Andrei Rybak, git; +Cc: Barret Rhoden Hi, On 19/06/21 05.11, Andrei Rybak wrote: > As can be seen in files "Documentation/blame-options.txt" and > "builtin/blame.c", the name of this configuration variable is > "blame.markUnblamableLines". > > Signed-off-by: Andrei Rybak <rybak.a.v@gmail.com> > --- > Documentation/config/blame.txt | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/Documentation/config/blame.txt b/Documentation/config/blame.txt > index 9468e8599c..4d047c1790 100644 > --- a/Documentation/config/blame.txt > +++ b/Documentation/config/blame.txt > @@ -27,7 +27,7 @@ blame.ignoreRevsFile:: > file names will reset the list of ignored revisions. This option will > be handled before the command line option `--ignore-revs-file`. > > -blame.markUnblamables:: > +blame.markUnblamableLines:: > Mark lines that were changed by an ignored revision that we could not > attribute to another commit with a '*' in the output of > linkgit:git-blame[1]. > That above is not typo fixing; that is correcting configuration variable name. PS: I have to download mbox of this thread and reply from it because I didn't receive the thread on my Thunderbird, regardless of CCing Git ML. -- An old man doll... just what I always wanted! - Clara ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH] blame: fix typo in documentation 2021-06-19 5:27 ` Bagas Sanjaya @ 2021-06-19 11:12 ` Andrei Rybak 2021-06-19 11:20 ` Bagas Sanjaya 2021-06-19 17:31 ` Felipe Contreras 1 sibling, 1 reply; 16+ messages in thread From: Andrei Rybak @ 2021-06-19 11:12 UTC (permalink / raw) To: Bagas Sanjaya; +Cc: Barret Rhoden, git On 19/06/2021 07:27, Bagas Sanjaya wrote: > On 19/06/21 05.11, Andrei Rybak wrote: >> As can be seen in files "Documentation/blame-options.txt" and >> "builtin/blame.c", the name of this configuration variable is >> "blame.markUnblamableLines". >> >> Signed-off-by: Andrei Rybak <rybak.a.v@gmail.com> >> --- >> Documentation/config/blame.txt | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/Documentation/config/blame.txt >> b/Documentation/config/blame.txt >> index 9468e8599c..4d047c1790 100644 >> --- a/Documentation/config/blame.txt >> +++ b/Documentation/config/blame.txt >> @@ -27,7 +27,7 @@ blame.ignoreRevsFile:: >> file names will reset the list of ignored revisions. This >> option will >> be handled before the command line option `--ignore-revs-file`. >> -blame.markUnblamables:: >> +blame.markUnblamableLines:: >> Mark lines that were changed by an ignored revision that we >> could not >> attribute to another commit with a '*' in the output of >> linkgit:git-blame[1]. >> > > That above is not typo fixing; that is correcting configuration variable > name. This issue in documentation of git-blame was introduced when the config option was added in commit 8934ac8c92 (blame: add config options for the output of ignored or unblamable lines, 2019-05-15). It first appeared in v6 of that series [1] where it was called blame.maskIgnoredUnblamables. In v7 [2] it got renamed to blame.markUnblamableLines. v7 of that series is the first time when incorrect blame.markUnblamables appears in Documentation/config/blame.txt. As far as I can tell, it's most probable that Barret Rhoden just missed one place when renaming this config option between v6 and v7. If we shouldn't refer to this patch as typofix, how about blame: fix name of blame.markUnblamableLines in docs or blame: correct name of config option in docs or config/blame.txt: correct name of blame.markUnblamableLines ? [1] https://lore.kernel.org/git/20190410162409.117264-5-brho@google.com/ [2] https://lore.kernel.org/git/20190515214503.77162-6-brho@google.com/ ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH] blame: fix typo in documentation 2021-06-19 11:12 ` Andrei Rybak @ 2021-06-19 11:20 ` Bagas Sanjaya 0 siblings, 0 replies; 16+ messages in thread From: Bagas Sanjaya @ 2021-06-19 11:20 UTC (permalink / raw) To: Andrei Rybak; +Cc: Barret Rhoden, git On 19/06/21 18.12, Andrei Rybak wrote: > If we shouldn't refer to this patch as typofix, how about > > blame: fix name of blame.markUnblamableLines in docs > > or > > blame: correct name of config option in docs > > or > > config/blame.txt: correct name of blame.markUnblamableLines > > ? > I prefer the second, because I prefer not to reveal actual config name so that readers must see the whole patch to see the name. -- An old man doll... just what I always wanted! - Clara ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH] blame: fix typo in documentation 2021-06-19 5:27 ` Bagas Sanjaya 2021-06-19 11:12 ` Andrei Rybak @ 2021-06-19 17:31 ` Felipe Contreras 1 sibling, 0 replies; 16+ messages in thread From: Felipe Contreras @ 2021-06-19 17:31 UTC (permalink / raw) To: Bagas Sanjaya, Andrei Rybak, git; +Cc: Barret Rhoden Bagas Sanjaya wrote: > That above is not typo fixing; that is correcting configuration variable > name. > > PS: I have to download mbox of this thread and reply from it because I > didn't receive the thread on my Thunderbird, regardless of CCing Git ML. This is something I've noticed; I can always see mails on the archive, however, I don't receive them on my inbox until much later. -- Felipe Contreras ^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH v2] blame: correct name of config option in docs 2021-06-18 22:11 [PATCH] blame: fix typo in documentation Andrei Rybak 2021-06-19 5:27 ` Bagas Sanjaya @ 2021-06-19 11:29 ` Andrei Rybak 2021-06-20 9:37 ` Bagas Sanjaya 2021-06-25 19:38 ` [PATCH v3 0/3] Config option names typofixes Andrei Rybak 1 sibling, 2 replies; 16+ messages in thread From: Andrei Rybak @ 2021-06-19 11:29 UTC (permalink / raw) To: git; +Cc: Barret Rhoden, Bagas Sanjaya As can be seen in files "Documentation/blame-options.txt" and "builtin/blame.c", the name of this configuration option is "blame.markUnblamableLines". Signed-off-by: Andrei Rybak <rybak.a.v@gmail.com> --- Thanks to Bagas Sanjaya for review. Changes since v1: - tweaked commit message Range-diff: 1: e32c298ee8 ! 1: 6ea64765e7 blame: fix typo in documentation @@ Metadata Author: Andrei Rybak <rybak.a.v@gmail.com> ## Commit message ## - blame: fix typo in documentation + blame: correct name of config option in docs As can be seen in files "Documentation/blame-options.txt" and - "builtin/blame.c", the name of this configuration variable is + "builtin/blame.c", the name of this configuration option is "blame.markUnblamableLines". Signed-off-by: Andrei Rybak <rybak.a.v@gmail.com> Documentation/config/blame.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/config/blame.txt b/Documentation/config/blame.txt index 9468e8599c..4d047c1790 100644 --- a/Documentation/config/blame.txt +++ b/Documentation/config/blame.txt @@ -27,7 +27,7 @@ blame.ignoreRevsFile:: file names will reset the list of ignored revisions. This option will be handled before the command line option `--ignore-revs-file`. -blame.markUnblamables:: +blame.markUnblamableLines:: Mark lines that were changed by an ignored revision that we could not attribute to another commit with a '*' in the output of linkgit:git-blame[1]. -- 2.32.0 ^ permalink raw reply related [flat|nested] 16+ messages in thread
* Re: [PATCH v2] blame: correct name of config option in docs 2021-06-19 11:29 ` [PATCH v2] blame: correct name of config option in docs Andrei Rybak @ 2021-06-20 9:37 ` Bagas Sanjaya 2021-06-21 16:57 ` Barret Rhoden 2021-06-25 19:38 ` [PATCH v3 0/3] Config option names typofixes Andrei Rybak 1 sibling, 1 reply; 16+ messages in thread From: Bagas Sanjaya @ 2021-06-20 9:37 UTC (permalink / raw) To: Andrei Rybak, git; +Cc: Barret Rhoden On 19/06/21 18.29, Andrei Rybak wrote: > As can be seen in files "Documentation/blame-options.txt" and > "builtin/blame.c", the name of this configuration option is > "blame.markUnblamableLines". > > Signed-off-by: Andrei Rybak <rybak.a.v@gmail.com> > --- > > Thanks to Bagas Sanjaya for review. Changes since v1: > > - tweaked commit message > > Range-diff: > > 1: e32c298ee8 ! 1: 6ea64765e7 blame: fix typo in documentation > @@ Metadata > Author: Andrei Rybak <rybak.a.v@gmail.com> > > ## Commit message ## > - blame: fix typo in documentation > + blame: correct name of config option in docs > > As can be seen in files "Documentation/blame-options.txt" and > - "builtin/blame.c", the name of this configuration variable is > + "builtin/blame.c", the name of this configuration option is > "blame.markUnblamableLines". > > Signed-off-by: Andrei Rybak <rybak.a.v@gmail.com> > > Documentation/config/blame.txt | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/Documentation/config/blame.txt b/Documentation/config/blame.txt > index 9468e8599c..4d047c1790 100644 > --- a/Documentation/config/blame.txt > +++ b/Documentation/config/blame.txt > @@ -27,7 +27,7 @@ blame.ignoreRevsFile:: > file names will reset the list of ignored revisions. This option will > be handled before the command line option `--ignore-revs-file`. > > -blame.markUnblamables:: > +blame.markUnblamableLines:: > Mark lines that were changed by an ignored revision that we could not > attribute to another commit with a '*' in the output of > linkgit:git-blame[1]. > Looks OK, thanks for contributing. Reviewed-by: Bagas Sanjaya <bagasdotme@gmail.com> -- An old man doll... just what I always wanted! - Clara ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH v2] blame: correct name of config option in docs 2021-06-20 9:37 ` Bagas Sanjaya @ 2021-06-21 16:57 ` Barret Rhoden 0 siblings, 0 replies; 16+ messages in thread From: Barret Rhoden @ 2021-06-21 16:57 UTC (permalink / raw) To: Bagas Sanjaya, Andrei Rybak, git On 6/20/21 5:37 AM, Bagas Sanjaya wrote: > On 19/06/21 18.29, Andrei Rybak wrote: >> As can be seen in files "Documentation/blame-options.txt" and >> "builtin/blame.c", the name of this configuration option is >> "blame.markUnblamableLines". >> >> Signed-off-by: Andrei Rybak <rybak.a.v@gmail.com> > Looks OK, thanks for contributing. > > Reviewed-by: Bagas Sanjaya <bagasdotme@gmail.com> > thanks for the fix. =) ^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH v3 0/3] Config option names typofixes 2021-06-19 11:29 ` [PATCH v2] blame: correct name of config option in docs Andrei Rybak 2021-06-20 9:37 ` Bagas Sanjaya @ 2021-06-25 19:38 ` Andrei Rybak 2021-06-25 19:38 ` [PATCH v3 1/3] blame: correct name of config option in docs Andrei Rybak ` (3 more replies) 1 sibling, 4 replies; 16+ messages in thread From: Andrei Rybak @ 2021-06-25 19:38 UTC (permalink / raw) To: git, Bagas Sanjaya, Barret Rhoden, Đoàn Trần Công Danh, Rafael Silva First patch in this series is the same as the v2 patch. Second patch fixes similar typos in mentions of config options in test messages. Third patch is just a fix for an unrelated typo I've stumbled upon. Andrei Rybak (3): blame: correct name of config option in docs t: fix typos in test messages git-worktree.txt: fix typo in example path Documentation/config/blame.txt | 2 +- Documentation/git-worktree.txt | 2 +- t/t4258-am-quoted-cr.sh | 2 +- t/t9001-send-email.sh | 2 +- 4 files changed, 4 insertions(+), 4 deletions(-) -- 2.32.0 ^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH v3 1/3] blame: correct name of config option in docs 2021-06-25 19:38 ` [PATCH v3 0/3] Config option names typofixes Andrei Rybak @ 2021-06-25 19:38 ` Andrei Rybak 2021-06-25 19:38 ` [PATCH v3 2/3] t: fix typos in test messages Andrei Rybak ` (2 subsequent siblings) 3 siblings, 0 replies; 16+ messages in thread From: Andrei Rybak @ 2021-06-25 19:38 UTC (permalink / raw) To: git, Bagas Sanjaya, Barret Rhoden, Đoàn Trần Công Danh, Rafael Silva As can be seen in files "Documentation/blame-options.txt" and "builtin/blame.c", the name of this configuration option is "blame.markUnblamableLines". Signed-off-by: Andrei Rybak <rybak.a.v@gmail.com> Reviewed-by: Bagas Sanjaya <bagasdotme@gmail.com> --- Documentation/config/blame.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/config/blame.txt b/Documentation/config/blame.txt index 9468e8599c..4d047c1790 100644 --- a/Documentation/config/blame.txt +++ b/Documentation/config/blame.txt @@ -27,7 +27,7 @@ blame.ignoreRevsFile:: file names will reset the list of ignored revisions. This option will be handled before the command line option `--ignore-revs-file`. -blame.markUnblamables:: +blame.markUnblamableLines:: Mark lines that were changed by an ignored revision that we could not attribute to another commit with a '*' in the output of linkgit:git-blame[1]. -- 2.32.0 ^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PATCH v3 2/3] t: fix typos in test messages 2021-06-25 19:38 ` [PATCH v3 0/3] Config option names typofixes Andrei Rybak 2021-06-25 19:38 ` [PATCH v3 1/3] blame: correct name of config option in docs Andrei Rybak @ 2021-06-25 19:38 ` Andrei Rybak 2021-06-25 19:38 ` [PATCH v3 3/3] git-worktree.txt: fix typo in example path Andrei Rybak 2021-06-26 11:50 ` [PATCH v3 0/3] Config option names typofixes Bagas Sanjaya 3 siblings, 0 replies; 16+ messages in thread From: Andrei Rybak @ 2021-06-25 19:38 UTC (permalink / raw) To: git, Bagas Sanjaya, Barret Rhoden, Đoàn Trần Công Danh, Rafael Silva Both in t4258 and in t9001, the code of the tests following shows the proper name for the configuration variables. So use the correct names in the test messages as well. Signed-off-by: Andrei Rybak <rybak.a.v@gmail.com> --- t/t4258-am-quoted-cr.sh | 2 +- t/t9001-send-email.sh | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/t/t4258-am-quoted-cr.sh b/t/t4258-am-quoted-cr.sh index fb5071f914..201915b45a 100755 --- a/t/t4258-am-quoted-cr.sh +++ b/t/t4258-am-quoted-cr.sh @@ -26,7 +26,7 @@ test_expect_success 'am --quoted-cr=strip' ' git diff --exit-code HEAD two ' -test_expect_success 'am with config mailinfo.quotecr=strip' ' +test_expect_success 'am with config mailinfo.quotedCr=strip' ' test_might_fail git am --abort && git reset --hard one && test_config mailinfo.quotedCr strip && diff --git a/t/t9001-send-email.sh b/t/t9001-send-email.sh index c66d9190de..9ec7d75f0f 100755 --- a/t/t9001-send-email.sh +++ b/t/t9001-send-email.sh @@ -1839,7 +1839,7 @@ test_expect_success $PREREQ 'sendemail.aliasfiletype=mailrc' ' grep "^!somebody@example\.org!$" commandline1 ' -test_expect_success $PREREQ 'sendemail.aliasfile=~/.mailrc' ' +test_expect_success $PREREQ 'sendemail.aliasesfile=~/.mailrc' ' clean_fake_sendmail && echo "alias sbd someone@example.org" >"$HOME/.mailrc" && git config --replace-all sendemail.aliasesfile "~/.mailrc" && -- 2.32.0 ^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PATCH v3 3/3] git-worktree.txt: fix typo in example path 2021-06-25 19:38 ` [PATCH v3 0/3] Config option names typofixes Andrei Rybak 2021-06-25 19:38 ` [PATCH v3 1/3] blame: correct name of config option in docs Andrei Rybak 2021-06-25 19:38 ` [PATCH v3 2/3] t: fix typos in test messages Andrei Rybak @ 2021-06-25 19:38 ` Andrei Rybak 2021-06-26 6:58 ` Carlo Marcelo Arenas Belón 2021-06-26 11:50 ` [PATCH v3 0/3] Config option names typofixes Bagas Sanjaya 3 siblings, 1 reply; 16+ messages in thread From: Andrei Rybak @ 2021-06-25 19:38 UTC (permalink / raw) To: git, Bagas Sanjaya, Barret Rhoden, Đoàn Trần Công Danh, Rafael Silva Signed-off-by: Andrei Rybak <rybak.a.v@gmail.com> --- Documentation/git-worktree.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/git-worktree.txt b/Documentation/git-worktree.txt index f1bb1fa5f5..66e67e6cbf 100644 --- a/Documentation/git-worktree.txt +++ b/Documentation/git-worktree.txt @@ -387,7 +387,7 @@ These annotations are: ------------ $ git worktree list /path/to/linked-worktree abcd1234 [master] -/path/to/locked-worktreee acbd5678 (brancha) locked +/path/to/locked-worktree acbd5678 (brancha) locked /path/to/prunable-worktree 5678abc (detached HEAD) prunable ------------ -- 2.32.0 ^ permalink raw reply related [flat|nested] 16+ messages in thread
* Re: [PATCH v3 3/3] git-worktree.txt: fix typo in example path 2021-06-25 19:38 ` [PATCH v3 3/3] git-worktree.txt: fix typo in example path Andrei Rybak @ 2021-06-26 6:58 ` Carlo Marcelo Arenas Belón 2021-06-26 10:07 ` Andrei Rybak 0 siblings, 1 reply; 16+ messages in thread From: Carlo Marcelo Arenas Belón @ 2021-06-26 6:58 UTC (permalink / raw) To: Andrei Rybak Cc: git, Bagas Sanjaya, Barret Rhoden, Đoàn Trần Công Danh, Rafael Silva, Matheus Tavares not sure if this patch really belongs in this specific patchset, but it might be worth also fixing the same typo in grep as shown by : --- >8 --- Subject: grep: fix worktree mis spelling 1d1729caeb (grep: replace grep_read_mutex by internal obj read lock, 2020-01-15) adds it as part of a comment fix it to the right spelling without the extra trailing 'e'. diff --git a/grep.c b/grep.c index 8f91af1cb0..f40c734565 100644 --- a/grep.c +++ b/grep.c @@ -1510,7 +1510,7 @@ static int fill_textconv_grep(struct repository *r, * * TODO: allowing text conversion to run in parallel with object * reading operations might increase performance in the multithreaded - * non-worktreee git-grep with --textconv. + * non-worktree git-grep with --textconv. */ obj_read_lock(); size = fill_textconv(r, driver, df, &buf); Signed-off-by: Carlo Marcelo Arenas Belón <carenas@gmail.com> ^ permalink raw reply related [flat|nested] 16+ messages in thread
* Re: [PATCH v3 3/3] git-worktree.txt: fix typo in example path 2021-06-26 6:58 ` Carlo Marcelo Arenas Belón @ 2021-06-26 10:07 ` Andrei Rybak 2021-06-29 1:12 ` Junio C Hamano 0 siblings, 1 reply; 16+ messages in thread From: Andrei Rybak @ 2021-06-26 10:07 UTC (permalink / raw) To: Carlo Marcelo Arenas Belón Cc: git, Bagas Sanjaya, Barret Rhoden, Đoàn Trần Công Danh, Rafael Silva, Matheus Tavares On 26/06/2021 08:58, Carlo Marcelo Arenas Belón wrote: > not sure if this patch really belongs in this specific patchset Yes, third patch is unrelated to the first two. >, but it > might be worth also fixing the same typo in grep as shown by : > > --- >8 --- > Subject: grep: fix worktree mis spelling s/mis spelling/misspelling/ > > 1d1729caeb (grep: replace grep_read_mutex by internal obj read lock, > 2020-01-15) adds it as part of a comment > > fix it to the right spelling without the extra trailing 'e'. > > diff --git a/grep.c b/grep.c > index 8f91af1cb0..f40c734565 100644 > --- a/grep.c > +++ b/grep.c > @@ -1510,7 +1510,7 @@ static int fill_textconv_grep(struct repository *r, > * > * TODO: allowing text conversion to run in parallel with object > * reading operations might increase performance in the multithreaded > - * non-worktreee git-grep with --textconv. > + * non-worktree git-grep with --textconv. I deliberately skipped this typo, because it is in a TODO comment, which I thought isn't worth fixing. I probably should have mentioned that in the patch I've sent. > */ > obj_read_lock(); > size = fill_textconv(r, driver, df, &buf); > > Signed-off-by: Carlo Marcelo Arenas Belón <carenas@gmail.com> > ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH v3 3/3] git-worktree.txt: fix typo in example path 2021-06-26 10:07 ` Andrei Rybak @ 2021-06-29 1:12 ` Junio C Hamano 0 siblings, 0 replies; 16+ messages in thread From: Junio C Hamano @ 2021-06-29 1:12 UTC (permalink / raw) To: Andrei Rybak Cc: Carlo Marcelo Arenas Belón, git, Bagas Sanjaya, Barret Rhoden, Đoàn Trần Công Danh, Rafael Silva, Matheus Tavares Andrei Rybak <rybak.a.v@gmail.com> writes: > On 26/06/2021 08:58, Carlo Marcelo Arenas Belón wrote: >> not sure if this patch really belongs in this specific patchset > > Yes, third patch is unrelated to the first two. > >>, but it >> might be worth also fixing the same typo in grep as shown by : >> --- >8 --- >> Subject: grep: fix worktree mis spelling > > s/mis spelling/misspelling/ > >> 1d1729caeb (grep: replace grep_read_mutex by internal obj read lock, >> 2020-01-15) adds it as part of a comment >> fix it to the right spelling without the extra trailing 'e'. >> diff --git a/grep.c b/grep.c >> index 8f91af1cb0..f40c734565 100644 >> --- a/grep.c >> +++ b/grep.c >> @@ -1510,7 +1510,7 @@ static int fill_textconv_grep(struct repository *r, >> * >> * TODO: allowing text conversion to run in parallel with object >> * reading operations might increase performance in the multithreaded >> - * non-worktreee git-grep with --textconv. >> + * non-worktree git-grep with --textconv. > > I deliberately skipped this typo, because it is in a TODO comment, which > I thought isn't worth fixing. I probably should have mentioned that in > the patch I've sent. All sound sensible. Will queue these three patches as-is. Thanks. ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH v3 0/3] Config option names typofixes 2021-06-25 19:38 ` [PATCH v3 0/3] Config option names typofixes Andrei Rybak ` (2 preceding siblings ...) 2021-06-25 19:38 ` [PATCH v3 3/3] git-worktree.txt: fix typo in example path Andrei Rybak @ 2021-06-26 11:50 ` Bagas Sanjaya 3 siblings, 0 replies; 16+ messages in thread From: Bagas Sanjaya @ 2021-06-26 11:50 UTC (permalink / raw) To: Andrei Rybak, git, Barret Rhoden, Đoàn Trần Công Danh, Rafael Silva On 26/06/21 02.38, Andrei Rybak wrote: > First patch in this series is the same as the v2 patch. Second patch fixes > similar typos in mentions of config options in test messages. Third patch is > just a fix for an unrelated typo I've stumbled upon. > > Andrei Rybak (3): > blame: correct name of config option in docs > t: fix typos in test messages > git-worktree.txt: fix typo in example path I suggest that the third patch should be on separate single patch, as it is unrelated to two other patches that correct config option names. Thanks. -- An old man doll... just what I always wanted! - Clara ^ permalink raw reply [flat|nested] 16+ messages in thread
end of thread, other threads:[~2021-06-29 1:12 UTC | newest] Thread overview: 16+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2021-06-18 22:11 [PATCH] blame: fix typo in documentation Andrei Rybak 2021-06-19 5:27 ` Bagas Sanjaya 2021-06-19 11:12 ` Andrei Rybak 2021-06-19 11:20 ` Bagas Sanjaya 2021-06-19 17:31 ` Felipe Contreras 2021-06-19 11:29 ` [PATCH v2] blame: correct name of config option in docs Andrei Rybak 2021-06-20 9:37 ` Bagas Sanjaya 2021-06-21 16:57 ` Barret Rhoden 2021-06-25 19:38 ` [PATCH v3 0/3] Config option names typofixes Andrei Rybak 2021-06-25 19:38 ` [PATCH v3 1/3] blame: correct name of config option in docs Andrei Rybak 2021-06-25 19:38 ` [PATCH v3 2/3] t: fix typos in test messages Andrei Rybak 2021-06-25 19:38 ` [PATCH v3 3/3] git-worktree.txt: fix typo in example path Andrei Rybak 2021-06-26 6:58 ` Carlo Marcelo Arenas Belón 2021-06-26 10:07 ` Andrei Rybak 2021-06-29 1:12 ` Junio C Hamano 2021-06-26 11:50 ` [PATCH v3 0/3] Config option names typofixes Bagas Sanjaya
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).