Re: about Starter project;implementing log --size in the print_commit function.

From: Junio C Hamano <gitster@pobox.com>
Date: Mar 25, 2015, 16:13 UTC
Message-ID: <xmqq1tkd3vij.fsf@gitster.dls.corp.google.com>
In-Reply-To: <CABJeaiqhL5eNPkaPpheG_Y47Rapj4ys3igHhFH6YuiroECVGDA@mail.gmail.com>

Shanti Swarup Tunga <b112041@iiit-bh.ac.in> writes:

Show 7 quoted lines

> I inserted a size variable of int type and calculated size in each
> cases of print_commit(). The function is
>
> static void print_commit(git_commit *commit)
> {
> ...
> }

See Documentation/SubmittingPatches and then also CodingGuidelines

Especially, explain:

 - what does this change try to help and how?
 - why is this patch a good way to achieve that goal?
 - how the users are expected to use the new feature?

in the proposed log message, run existing tests to make sure your change did not break the existing system, add tests to make sure you would notice if somebody else broke the feature you are adding in the future, and describe the new feature in the documentation.

> Will that approach is going to work for the following problem.?

Was your message cut short? We cannot read "the following" problem here...