threads / patch / 60369

patchtypo: fix the typo 'neeed' into 'needed' in the comment under merge-ort.c

Subject: [PATCH] typo: fix the typo 'neeed' into 'needed' in the comment under merge-ort.c

## tl;dr

3 messages between Oct 15, 2023 and Oct 22, 2023. Diffs are folded; open one to read it.

replies: 2people: 1as markdown or json

Wangchangxin via GitGitGadget· Oct 15, 2023, 14:08 UTC · lore

From: foril <1571825323@qq.com>

Signed-off-by: 王常新 (Wang Changxin) <foril@foril.space>
---
    typo: fix the typo 'neeed' into 'needed' in the comment under merge-o…
    
    the comments on line 2039 under merge-ort.c should be :
    this is needed if we have content merges of content merges rather than
    this is neeed if we have content merges of content merges
    
    fix the typo

Published-As: https://github.com/gitgitgadget/git/releases/tag/pr-git-1592%2FforiLLL%2Fcomment_patch-v1
Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-git-1592/foriLLL/comment_patch-v1
Pull-Request: https://github.com/git/git/pull/1592

 merge-ort.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Show changes to merge-ort.c +1 −1

diff --git a/merge-ort.c b/merge-ort.c
index 7857ce9fbd1..aee6f7d8173 100644
--- a/merge-ort.c
+++ b/merge-ort.c
@@ -2036,7 +2036,7 @@ static int handle_content_merge(struct merge_options *opt,
 	 * the three blobs to merge on various sides of history.
 	 *
 	 * extra_marker_size is the amount to extend conflict markers in
-	 * ll_merge; this is neeed if we have content merges of content
+	 * ll_merge; this is needed if we have content merges of content
 	 * merges, which happens for example with rename/rename(2to1) and
 	 * rename/add conflicts.
 	 */

base-commit: a9ecda2788e229afc9b611acaa26d0d9d4da53ed

-- 
gitgitgadget

Wangchangxin via GitGitGadget· Oct 19, 2023, 12:40 UTC · re: Wangchangxin via GitGitGadget · lore

[PATCH v2] merge-ort.c: fix typo 'neeed' to 'needed'

From: foril <1571825323@qq.com>

Signed-off-by: 王常新 (Wang Changxin) <foril@foril.space>
---
    typo: fix the typo 'neeed' into 'needed' in the comment under merge-o…
    
    the comments on line 2039 under merge-ort.c should be :
    this is needed if we have content merges of content merges rather than
    this is neeed if we have content merges of content merges
    
    fix the typo

Published-As: https://github.com/gitgitgadget/git/releases/tag/pr-git-1592%2FforiLLL%2Fcomment_patch-v2
Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-git-1592/foriLLL/comment_patch-v2
Pull-Request: https://github.com/git/git/pull/1592

Range-diff vs v1:

 1:  737eccd2811 ! 1:  3934ee6b684 typo: fix the typo 'neeed' into 'needed' in the comment under merge-ort.c
     @@ Metadata
      Author: foril <1571825323@qq.com>
      
       ## Commit message ##
     -    typo: fix the typo 'neeed' into 'needed' in the comment under merge-ort.c
     +    merge-ort.c: fix typo 'neeed' to 'needed'
      
          Signed-off-by: 王常新 (Wang Changxin) <foril@foril.space>

 merge-ort.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Show changes to merge-ort.c +1 −1

diff --git a/merge-ort.c b/merge-ort.c
index 7857ce9fbd1..aee6f7d8173 100644
--- a/merge-ort.c
+++ b/merge-ort.c
@@ -2036,7 +2036,7 @@ static int handle_content_merge(struct merge_options *opt,
 	 * the three blobs to merge on various sides of history.
 	 *
 	 * extra_marker_size is the amount to extend conflict markers in
-	 * ll_merge; this is neeed if we have content merges of content
+	 * ll_merge; this is needed if we have content merges of content
 	 * merges, which happens for example with rename/rename(2to1) and
 	 * rename/add conflicts.
 	 */

base-commit: a9ecda2788e229afc9b611acaa26d0d9d4da53ed

-- 
gitgitgadget

Wangchangxin via GitGitGadget· Oct 22, 2023, 02:46 UTC · re: Wangchangxin via GitGitGadget · lore

[PATCH v3] merge-ort.c: fix typo 'neeed' to 'needed'

From: =?UTF-8?q?=E7=8E=8B=E5=B8=B8=E6=96=B0?= <wchangxin824@gmail.com>

Signed-off-by: 王常新 (Wang Changxin) <wchangxin824@gmail.com>
---
    typo: fix the typo 'neeed' into 'needed' in the comment under merge-o…
    
    the comments on line 2039 under merge-ort.c should be :
    this is needed if we have content merges of content merges rather than
    this is neeed if we have content merges of content merges
    
    fix the typo

Published-As: https://github.com/gitgitgadget/git/releases/tag/pr-git-1592%2FforiLLL%2Fcomment_patch-v3
Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-git-1592/foriLLL/comment_patch-v3
Pull-Request: https://github.com/git/git/pull/1592

Range-diff vs v2:

 1:  3934ee6b684 ! 1:  9320154b91a merge-ort.c: fix typo 'neeed' to 'needed'
     @@
       ## Metadata ##
     -Author: foril <1571825323@qq.com>
     +Author: 王常新 <wchangxin824@gmail.com>
      
       ## Commit message ##
          merge-ort.c: fix typo 'neeed' to 'needed'
      
     -    Signed-off-by: 王常新 (Wang Changxin) <foril@foril.space>
     +    Signed-off-by: 王常新 (Wang Changxin) <wchangxin824@gmail.com>
      
       ## merge-ort.c ##
      @@ merge-ort.c: static int handle_content_merge(struct merge_options *opt,

 merge-ort.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Show changes to merge-ort.c +1 −1

diff --git a/merge-ort.c b/merge-ort.c
index 7857ce9fbd1..aee6f7d8173 100644
--- a/merge-ort.c
+++ b/merge-ort.c
@@ -2036,7 +2036,7 @@ static int handle_content_merge(struct merge_options *opt,
 	 * the three blobs to merge on various sides of history.
 	 *
 	 * extra_marker_size is the amount to extend conflict markers in
-	 * ll_merge; this is neeed if we have content merges of content
+	 * ll_merge; this is needed if we have content merges of content
 	 * merges, which happens for example with rename/rename(2to1) and
 	 * rename/add conflicts.
 	 */

base-commit: a9ecda2788e229afc9b611acaa26d0d9d4da53ed

-- 
gitgitgadget