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