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

Re: [PATCH 2/2] Add support for GIT_CEILING_DIRS

From
Kevin Ballard <kevin@sb.org>
Date
May 14, 2008, 16:51 UTC
Message-ID
<D2EA6017-4607-4A55-8C72-CD2B772CAAE4@sb.org>
In-Reply-To
<alpine.DEB.1.00.0805140903560.30431@racer>
On May 14, 2008, at 3:23 AM, Johannes Schindelin wrote:
Show 11 quoted lines
>> @@ -414,6 +415,37 @@ const char *setup_git_directory_gently(int  
>> *nongit_ok)
>> 	if (!getcwd(cwd, sizeof(cwd)-1))
>> 		die("Unable to read current working directory");
>>
>> +	// Compute min_offset based on GIT_CEILING_DIRS.
>
> We do not like C99 style comments.  Remember, there are people who  
> compile
> Git on something else than the super-latest Linux with cutting-edge  
> GCC.

Out of curiosity, what environment these days doesn't allow C99 comments? I think I started programming roughly 10 years ago (granted, I didn't start with C, but I did C programming at least 8 years ago, if not earlier) and every environment I've ever used has supported those comments even in C89 mode.

I mean, I fully understand that git wants to be portable across all platforms, but it still makes me wonder if there's any environment that anybody actually uses these days that doesn't understand those comments when compiling C.

-- 
Kevin Ballard
http://kevin.sb.org
kevin@sb.org
http://www.tildesoft.com
Previous: Johannes SixtNext: Johannes Schindelin
Message 4 of 6 in “Add support for GIT_CEILING_DIRS”
  1. 2/2 Add support for GIT_CEILING_DIRSDavid Reiss, May 14, 2008
  2. Johannes SchindelinMay 14, 2008
  3. Johannes SixtMay 14, 2008
  4. Kevin BallardMay 14, 2008
  5. Johannes SchindelinMay 14, 2008
  6. Kevin BallardMay 14, 2008

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.