git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH v4] commit.c: use skip_prefix() instead of starts_with()

From
Junio C Hamano <gitster@pobox.com>
Date
Mar 5, 2014, 22:23 UTC
Message-ID
<xmqqd2i02sku.fsf@gitster.dls.corp.google.com>
In-Reply-To
<CAEc54XB3hdsJcO4QFuqFHi+mZtZDrV2wFdMQ1uHiPrxzfo99jw@mail.gmail.com>
tanay abhra <tanayabh@gmail.com> writes:
Show 22 quoted lines
> On Wed, Mar 5, 2014 at 3:41 AM, Junio C Hamano <gitster@pobox.com> wrote:
>
>> Junio C Hamano <gitster@pobox.com> writes:
>>
>> >> +            found = skip_prefix(buf, sigcheck_gpg_status[i].check + 1);
>> >> +            if(!found) {
>>
>> Missing SP between the control keyword and parenthesized expression
>> the keyword uses.
>>
>> I've fixed this (and the broken indentation) locally and queued the
>> result to 'pu', so no need to resend to correct this one.
>>
>> Thanks.
>>
>>
> Sorry about the indentation, it was due to not setting the tab to eight
> spaces. I found your mail late, so I had already
> sent a revised patch [1]. Please ignore that mail.
>
> Also , what should be my next step ,should I present a rough draft of a
> proposal , or tackle other bugs on the mailing list?

As far as I, as the maintainer of the project, am concerned [*1*], we are done and I expect/require nothing more from you on this change.

The MicroProject page says:
    ... If you've already done a microproject and are itching to do
    more, then get involved in other ways, like finding and fixing
    other problems in the code, or improving the documentation or
    code comments, or helping to review other people's patches on
    the mailing list, or answering questions on the mailing list or
    in IRC, or writing new tests, etc., etc. In short, start doing
    things that other Git developers do!

FYI, [GSoC 2014 timeline] (Google for it) tells us that would-be students are supposed to be discussing project ideas with their mentoring organizations now, in preparation for the actual student application that begins on Mar 10 and ends on Mar 21.

[Footnote]

*1* I should mention that I am not involved in GSoC administration and student selection for the Git project.

Previous: Tanay Abhra
Message 5 of 5 in “commit.c: use skip_prefix() instead of starts_with()”
  1. commit.c: use skip_prefix() instead of starts_with()Tanay Abhra, Mar 4, 2014
  2. Junio C HamanoMar 4, 2014
  3. Junio C HamanoMar 4, 2014
  4. Tanay AbhraMar 4, 2014
  5. Junio C HamanoMar 5, 2014

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.