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

Re: update-cache ./test.c

From
TGThomas Glanzmann <sithglan@stud.uni-erlangen.de>
Date
May 1, 2005, 09:04 UTC
Message-ID
<20050501090437.GK26663@cip.informatik.uni-erlangen.de>
In-Reply-To
<20050501085922.GC30692@xi.wantstofly.org>
Hello,
> --- git/update-cache.c.orig	2005-05-01 10:56:17.859313581 +0200
> +++ git/update-cache.c	2005-05-01 10:57:31.634897508 +0200
> @@ -328,6 +328,7 @@
>  	for (i = 1 ; i < argc; i++) {
>  		char *path = argv[i];
> +		char *_path;

I think there is no need to introduce an extra variable. Just increase path by two. I knew that it is easy to fix in the code, but I think the 'logic' should go into the frontend not in the backend. But this one is really easy to fix. Maybe regenerate a patch and sign it off, maybe it will be included.

Greetings,
	Thomas
Previous: Lennert BuytenhekNext: Junio C Hamano
Message 5 of 7 in “update-cache ./test.c”
  1. Lennert BuytenhekMay 1, 2005
  2. Thomas GlanzmannMay 1, 2005
  3. Lennert BuytenhekMay 1, 2005
  4. Lennert BuytenhekMay 1, 2005
  5. Thomas GlanzmannMay 1, 2005
  6. Junio C HamanoMay 1, 2005
  7. David GreavesMay 1, 2005

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.