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

Re: [PATCH 1/2] Function stripspace now gets a buffer instead file descriptors.

From
Junio C Hamano <gitster@pobox.com>
Date
Jul 12, 2007, 00:03 UTC
Message-ID
<7v8x9m1oac.fsf@assigned-by-dhcp.cox.net>
In-Reply-To
<1b46aba20707111641y615ff82dpf2da4afab2b17d58@mail.gmail.com>
"Carlos Rica" <jasampler@gmail.com> writes:
Show 8 quoted lines
> Now the function cleanup is not removing spaces at all, but only
> counting where the line of text ends.
>
> Before, in the previous version, the function cleanup was taking a
> string as argument, and then it needed to modify that string.  Now, it
> just returns the new length, "not counting" the spaces and newline at
> the end of the buffer passed. Its name and comments then could be
> different, but I didn't know which ones.

Ah, you are right. I misread the patch --- cleanup() does not even touch the buffer.

Previous: Carlos RicaNext: Carlos Rica
Message 6 of 7 in “Function stripspace now gets a buffer instead file descriptors.”
  1. 1/2 Function stripspace now gets a buffer instead file descriptors.Carlos Rica, Jul 11, 2007
  2. Johannes SchindelinJul 11, 2007
  3. Junio C HamanoJul 11, 2007
  4. Bill LearJul 11, 2007
  5. Carlos RicaJul 11, 2007
  6. Junio C HamanoJul 12, 2007
  7. Carlos RicaJul 12, 2007

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.