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

Re: [PATCH] Make git-mailinfo strip whitespace from the start of the mail file.

From
Junio C Hamano <gitster@pobox.com>
Date
Nov 2, 2007, 08:53 UTC
Message-ID
<7v8x5h58qj.fsf@gitster.siamese.dyndns.org>
In-Reply-To
<1193951139-2312-1-git-send-email-Simon.Sasburg@gmail.com>
Simon Sasburg <simon.sasburg@gmail.com> writes:
Show 9 quoted lines
> @@ -935,6 +936,11 @@ static int mailinfo(FILE *in, FILE *out, int ks, const char *encoding,
>  	p_hdr_data = xcalloc(MAX_HDR_PARSED, sizeof(char *));
>  	s_hdr_data = xcalloc(MAX_HDR_PARSED, sizeof(char *));
>  
> +	do {
> +		peek = fgetc(in);
> +	} while (peek == ' ' || peek == '\r' || peek == '\n');
> +	ungetc(peek, in);
> +
I wonder why this is not using isspace(peek).
Previous: Simon SasburgNext: Simon Sasburg
Message 6 of 7 in “Make git-mailinfo strip whitespace from the start of the mail file.”
  1. Make git-mailinfo strip whitespace from the start of the mail file.Simon Sasburg, Nov 1, 2007
  2. Junio C HamanoNov 1, 2007
  3. Simon SasburgNov 1, 2007
  4. Junio C HamanoNov 1, 2007
  5. Make git-mailsplit strip whitespace from the start of the mailbox file.Simon Sasburg, Nov 1, 2007
  6. Junio C HamanoNov 2, 2007
  7. Make git-mailsplit strip whitespace from the start of the mailbox file.Simon Sasburg, Nov 4, 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.