Re: [PATCH] [PATCH] commit.c: Replace starts_with() with skip_prefix()

2014-03-04 Thread Eric Sunshine
On Tue, Mar 4, 2014 at 5:27 PM, Eric Sunshine wrote: > On Tue, Mar 4, 2014 at 10:54 AM, Karthik Nayak wrote: >> diff --git a/commit.c b/commit.c >> index 6bf4fe0..71a03e3 100644 >> --- a/commit.c >> +++ b/commit.c >> @@ -,11 +1114,11 @@ int parse_signed_commit(const unsigned char *sha1, >>

Re: [PATCH] [PATCH] commit.c: Replace starts_with() with skip_prefix()

2014-03-04 Thread Eric Sunshine
On Tue, Mar 4, 2014 at 5:27 PM, Eric Sunshine wrote: > On Tue, Mar 4, 2014 at 10:54 AM, Karthik Nayak wrote: >> diff --git a/commit.c b/commit.c >> index 6bf4fe0..71a03e3 100644 >> --- a/commit.c >> +++ b/commit.c >> @@ -1193,10 +1196,8 @@ static void parse_gpg_output(struct signature_check >> *

Re: [PATCH] [PATCH] commit.c: Replace starts_with() with skip_prefix()

2014-03-04 Thread Eric Sunshine
On Tue, Mar 4, 2014 at 12:58 PM, karthik nayak wrote: > Hey Tanay, > > 1. Yes just getting used to git send-email now, should follow that from now > 2. I thought it shouldn't be a part of the commit, so i put it after > the last --- > 3. I did have a thought on your lines also , but concluded to i

Re: [PATCH] [PATCH] commit.c: Replace starts_with() with skip_prefix()

2014-03-04 Thread Eric Sunshine
Thanks for the resend. Etiquette on this list is to cc: people who commented on previous versions of the submission. As Tanay already mentioned, use [PATCH vN] in the subject where N is the version number of this attempt. The -v option of "git format-email" can help. More below. On Tue, Mar 4, 20

Re: [PATCH] [PATCH] commit.c: Replace starts_with() with skip_prefix()

2014-03-04 Thread karthik nayak
Hey Tanay, 1. Yes just getting used to git send-email now, should follow that from now 2. I thought it shouldn't be a part of the commit, so i put it after the last --- 3. I did have a thought on your lines also , but concluded to it being more advantageous, you might be right though Nice to hear

Re: [PATCH] [PATCH] commit.c: Replace starts_with() with skip_prefix()

2014-03-04 Thread Tanay Abhra
Karthik Nayak gmail.com> writes: > > In record_author_date() : > Replace "buf + strlen(author )" by skip_prefix(), which is > saved in a new "const char" variable "indent_line". > > In parse_signed_commit() : > Replace "line + gpg_sig_header_len" by skip_prefix(), which > is saved in a new "co

[PATCH] [PATCH] commit.c: Replace starts_with() with skip_prefix()

2014-03-04 Thread Karthik Nayak
In record_author_date() : Replace "buf + strlen(author )" by skip_prefix(), which is saved in a new "const char" variable "indent_line". In parse_signed_commit() : Replace "line + gpg_sig_header_len" by skip_prefix(), which is saved in a new "const char" variable "indent_line". In parse_gpg_outpu

Re: [PATCH] commit.c: Replace starts_with() with skip_prefix()

2014-03-03 Thread Eric Sunshine
On Mon, Mar 3, 2014 at 10:23 AM, karthik nayak wrote: > Hello Eric, > Thanks for Pointing out everything, i had a thorough look and fixed a > couple of things. > Here is an Updated Patch. > - Removed unnecessary code and variables. > - Replaced all instances of starts_with() with skip_prefix() Th

Re: [PATCH] commit.c: Replace starts_with() with skip_prefix()

2014-03-03 Thread karthik nayak
Hello Eric, Thanks for Pointing out everything, i had a thorough look and fixed a couple of things. Here is an Updated Patch. - Removed unnecessary code and variables. - Replaced all instances of starts_with() with skip_prefix() Replace starts_with() with skip_prefix() for better reading purposes.

Re: [PATCH] commit.c: Replace starts_with() with skip_prefix()

2014-03-03 Thread Eric Sunshine
Thanks for the submission. Some commentary below to expose you to the review process on this project... On Mon, Mar 3, 2014 at 2:47 AM, Karthik Nayak wrote: > Replace with skip_prefix(), which uses the inbuilt function > strcmp() to compare. Explaining the purpose of the change is indeed importa

[PATCH] commit.c: Replace starts_with() with skip_prefix()

2014-03-02 Thread Karthik Nayak
Replace with skip_prefix(), which uses the inbuilt function strcmp() to compare. Other Places were this can be implemented: commit.c : line 1117 commit.c : line 1197 Signed-off-by: Karthik Nayak --- commit.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/commit.c b/commit