git@vger.kernel.org mailing list mirror (one of many)
 help / color / mirror / code / Atom feed
* [PATCH] comment: fix a typo in the comment
@ 2017-07-05 17:03 Kaartic Sivaraam
  2017-07-05 18:18 ` Junio C Hamano
  0 siblings, 1 reply; 7+ messages in thread
From: Kaartic Sivaraam @ 2017-07-05 17:03 UTC (permalink / raw)
  To: gitster; +Cc: git

---
 Though very trivial, I wanted to correct this as I didn't
 want to ignore it after seeing it.

 builtin/commit.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/builtin/commit.c b/builtin/commit.c
index 8d1cac0..aff6bf7 100644
--- a/builtin/commit.c
+++ b/builtin/commit.c
@@ -984,7 +984,7 @@ static int rest_is_empty(struct strbuf *sb, int start)
 	int i, eol;
 	const char *nl;
 
-	/* Check if the rest is just whitespace and Signed-of-by's. */
+	/* Check if the rest is just whitespace and Signed-off-by's. */
 	for (i = start; i < sb->len; i++) {
 		nl = memchr(sb->buf + i, '\n', sb->len - i);
 		if (nl)
-- 
2.7.4


^ permalink raw reply related	[flat|nested] 7+ messages in thread

* Re: [PATCH] comment: fix a typo in the comment
  2017-07-05 17:03 [PATCH] comment: fix a typo in the comment Kaartic Sivaraam
@ 2017-07-05 18:18 ` Junio C Hamano
  2017-07-06  3:17   ` Kaartic Sivaraam
  2017-07-06  3:19   ` [PATCH] comment: " Kaartic Sivaraam
  0 siblings, 2 replies; 7+ messages in thread
From: Junio C Hamano @ 2017-07-05 18:18 UTC (permalink / raw)
  To: Kaartic Sivaraam; +Cc: git

Kaartic Sivaraam <kaarticsivaraam91196@gmail.com> writes:

> ---
>  Though very trivial, I wanted to correct this as I didn't
>  want to ignore it after seeing it.

Thanks for sharp eyes.  Sign-off?  (or Sign-of? ;-))

>
>  builtin/commit.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/builtin/commit.c b/builtin/commit.c
> index 8d1cac0..aff6bf7 100644
> --- a/builtin/commit.c
> +++ b/builtin/commit.c
> @@ -984,7 +984,7 @@ static int rest_is_empty(struct strbuf *sb, int start)
>  	int i, eol;
>  	const char *nl;
>  
> -	/* Check if the rest is just whitespace and Signed-of-by's. */
> +	/* Check if the rest is just whitespace and Signed-off-by's. */
>  	for (i = start; i < sb->len; i++) {
>  		nl = memchr(sb->buf + i, '\n', sb->len - i);
>  		if (nl)

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] comment: fix a typo in the comment
  2017-07-05 18:18 ` Junio C Hamano
@ 2017-07-06  3:17   ` Kaartic Sivaraam
  2017-07-06 14:47     ` SZEDER Gábor
  2017-07-06  3:19   ` [PATCH] comment: " Kaartic Sivaraam
  1 sibling, 1 reply; 7+ messages in thread
From: Kaartic Sivaraam @ 2017-07-06  3:17 UTC (permalink / raw)
  To: Junio C Hamano; +Cc: git

On Wed, 2017-07-05 at 11:18 -0700, Junio C Hamano wrote:
> Kaartic Sivaraam <kaarticsivaraam91196@gmail.com> writes:
> 
> > ---
> >  Though very trivial, I wanted to correct this as I didn't
> >  want to ignore it after seeing it.
> 
> Thanks for sharp eyes.  Sign-off?  (or Sign-of? ;-))
> 
I should also thank you for your sharp eyes! BTW, this won't repeat
again as I have made 'git' worry about adding the signed-off-by to the
commits I do on my local version of git.git :)

-- 
Kaartic

^ permalink raw reply	[flat|nested] 7+ messages in thread

* [PATCH] comment: fix a typo in the comment
  2017-07-05 18:18 ` Junio C Hamano
  2017-07-06  3:17   ` Kaartic Sivaraam
@ 2017-07-06  3:19   ` Kaartic Sivaraam
  1 sibling, 0 replies; 7+ messages in thread
From: Kaartic Sivaraam @ 2017-07-06  3:19 UTC (permalink / raw)
  To: gitster; +Cc: git

Signed-off-by: Kaartic Sivaraam <kaarticsivaraam91196@gmail.com>
---
 builtin/commit.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/builtin/commit.c b/builtin/commit.c
index 8d1cac062..aff6bf7aa 100644
--- a/builtin/commit.c
+++ b/builtin/commit.c
@@ -984,7 +984,7 @@ static int rest_is_empty(struct strbuf *sb, int start)
 	int i, eol;
 	const char *nl;
 
-	/* Check if the rest is just whitespace and Signed-of-by's. */
+	/* Check if the rest is just whitespace and Signed-off-by's. */
 	for (i = start; i < sb->len; i++) {
 		nl = memchr(sb->buf + i, '\n', sb->len - i);
 		if (nl)
-- 
2.11.0


^ permalink raw reply related	[flat|nested] 7+ messages in thread

* Re: [PATCH] comment: fix a typo in the comment
  2017-07-06  3:17   ` Kaartic Sivaraam
@ 2017-07-06 14:47     ` SZEDER Gábor
  2017-07-06 16:04       ` Junio C Hamano
  0 siblings, 1 reply; 7+ messages in thread
From: SZEDER Gábor @ 2017-07-06 14:47 UTC (permalink / raw)
  To: Kaartic Sivaraam, Junio C Hamano; +Cc: SZEDER Gábor, git

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset=UTF-8, Size: 456 bytes --]

> On Wed, 2017-07-05 at 11:18 -0700, Junio C Hamano wrote:
> > Kaartic Sivaraam <kaarticsivaraam91196@gmail.com> writes:
> > 
> > > ---
> > >  Though very trivial, I wanted to correct this as I didn't
> > >  want to ignore it after seeing it.
> > 
> > Thanks for sharp eyes.  Sign-off?  (or Sign-of? ;-))
> > 
> I should also thank you for your sharp eyes!

Speaking of sharp eyes...  That Subject: line really needs a
s/comment:/commit:/, doesn't it? :)


^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] comment: fix a typo in the comment
  2017-07-06 14:47     ` SZEDER Gábor
@ 2017-07-06 16:04       ` Junio C Hamano
  2017-07-07 16:02         ` [PATCH] commit: " Kaartic Sivaraam
  0 siblings, 1 reply; 7+ messages in thread
From: Junio C Hamano @ 2017-07-06 16:04 UTC (permalink / raw)
  To: SZEDER Gábor; +Cc: Kaartic Sivaraam, git

SZEDER Gábor <szeder.dev@gmail.com> writes:

> Speaking of sharp eyes...  That Subject: line really needs a
> s/comment:/commit:/, doesn't it? :)

Surely it does.

^ permalink raw reply	[flat|nested] 7+ messages in thread

* [PATCH] commit: fix a typo in the comment
  2017-07-06 16:04       ` Junio C Hamano
@ 2017-07-07 16:02         ` Kaartic Sivaraam
  0 siblings, 0 replies; 7+ messages in thread
From: Kaartic Sivaraam @ 2017-07-07 16:02 UTC (permalink / raw)
  To: gitster; +Cc: git

Signed-off-by: Kaartic Sivaraam <kaarticsivaraam91196@gmail.com>
---
 builtin/commit.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/builtin/commit.c b/builtin/commit.c
index 8d1cac062..aff6bf7aa 100644
--- a/builtin/commit.c
+++ b/builtin/commit.c
@@ -984,7 +984,7 @@ static int rest_is_empty(struct strbuf *sb, int start)
 	int i, eol;
 	const char *nl;
 
-	/* Check if the rest is just whitespace and Signed-of-by's. */
+	/* Check if the rest is just whitespace and Signed-off-by's. */
 	for (i = start; i < sb->len; i++) {
 		nl = memchr(sb->buf + i, '\n', sb->len - i);
 		if (nl)
-- 
2.13.2.879.g2ab69f31a.dirty


^ permalink raw reply related	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2017-07-07 16:02 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-07-05 17:03 [PATCH] comment: fix a typo in the comment Kaartic Sivaraam
2017-07-05 18:18 ` Junio C Hamano
2017-07-06  3:17   ` Kaartic Sivaraam
2017-07-06 14:47     ` SZEDER Gábor
2017-07-06 16:04       ` Junio C Hamano
2017-07-07 16:02         ` [PATCH] commit: " Kaartic Sivaraam
2017-07-06  3:19   ` [PATCH] comment: " Kaartic Sivaraam

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).