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

Re: [PATCH 2/2][GSoC 2014] diff: used is_dot_or_dotdot() in code

From
Eric Sunshine <sunshine@sunshineco.com>
Date
Mar 19, 2014, 04:08 UTC
Message-ID
<CAPig+cQJm7MoftaGd5=p-ooLVQQguzhwsYHNFgjjzywyq-UC=A@mail.gmail.com>
In-Reply-To
<CAM+=D-CO9A4hYb+9tVf_cAOMqkTm3taBLPp0eHtFkuMUCFw2ng@mail.gmail.com>
On Tue, Mar 18, 2014 at 11:58 PM, Brian Bourn <ba.bourn@gmail.com> wrote:
Show 28 quoted lines
> On Tue, Mar 18, 2014 at 11:45 PM, Eric Sunshine <sunshine@sunshineco.com>
> wrote:
>> On Tue, Mar 18, 2014 at 9:30 PM, babourn <ba.bourn@gmail.com> wrote:
>> > Subject: diff: used is_dot_or_dotdot() in code
>> > Signed-off-by: Brian Bourn <bab2177@columbia.edu>
>> > ---
>> >  diff-no-index.c | 2 +-
>> >  1 file changed, 1 insertion(+), 1 deletion(-)
>> >
>> > diff --git a/diff-no-index.c b/diff-no-index.c
>> > index ba915af..44cce25 100644
>> > --- a/diff-no-index.c
>> > +++ b/diff-no-index.c
>> > @@ -26,7 +26,7 @@ static int read_directory_contents(const char *path,
>> > struct string_list *list)
>> >   return error("Could not open directory %s", path);
>> >
>> >   while ((e = readdir(dir)))
>> > - if (strcmp(".", e->d_name) && strcmp("..", e->d_name))
>> > + if (!is_dot_or_dotdot(e->d_name))
>>
>> The patch is severely whitespace-damaged. (Did you post it through
>> Nabble?)
>
>    I did post through Nabble, My email with the patch didn't seem to be
> going through.
>    should I keep trying to resend it through email to undo the whitspace
> damage?

It's probably not necessary to try resending this version of the patch since you'll (hopefully) be sending a newer version which takes reviewer comments into consideration.

What method are you using to send the patches? git send-email? Something other? This particular mailing list rejects HTML-formatted messages, so that could be the culprit if you pasted the patch into your email client. It's a good idea to try sending patches to yourself via "git send-email". If you can get that to work successfully, then they should be accepted by the mailing list when sent via the same mechanism.

Previous: Eric Sunshine
Message 5 of 5 in “[GSoC 2014]diff: Imported dir.h and renamed read_directory()”
  1. [GSoC 2014]diff: Imported dir.h and renamed read_directory()Brian Bourn, Mar 19, 2014
  2. Eric SunshineMar 19, 2014
  3. 2/2 [GSoC 2014] diff: used is_dot_or_dotdot() in codebabourn, Mar 19, 2014
  4. Eric SunshineMar 19, 2014
  5. Eric SunshineMar 19, 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.