Re: [PATCH 12/13] Add bundle transport

!MAILaRCHIVE_VOTE_RePLACE
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: Pierre Habouzit <madcoder@...>
Cc: Johannes Schindelin <Johannes.Schindelin@...>, Junio C Hamano <junkio@...>, <git@...>
Date: Tuesday, September 11, 2007 - 1:46 pm

On Tue, 11 Sep 2007, Pierre Habouzit wrote:

But it does.

The "latest" being the compiler.

A lot of people have old compilers, possibly even ones where there *are* 
no compiler versions that support C99.


The best way to handle that is to make sure you use ANSI C (which we *do* 
depend on), and strict prototypes everywhere. Then, any breakage will not 
be silent, because different functions will have different prototypes.

Yes, C99 structure initializers are a good thing, and we use them in the 
linux kernel, but the kernel can make more assumptions about the compiler 
than a random user land tool can. So git should be more conservative than 
that.

It wasn't that long ago that people avoided even ANSI C, on the grounds 
that some (totally broken crap) OS's shipped with just a K&R compiler by 
default, and you had to pay for the "fancy" ANSI C compiler. Now, *that* 
kind of ass-backwards compatibility is just stupid in this day and age 
(especially since the advantages of ANSI over K&R are just so huge), but 
at assuming everybody has C99 is not really realistic yet, and the things 
C99 brings aren't quite worth breaking old compiler setups for.

(But we may be getting close to being able to assume C99. It's getting 
pretty common).

			Linus
-
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH 12/13] Add bundle transport, Johannes Schindelin, (Mon Sep 10, 11:03 pm)
Re: [PATCH 12/13] Add bundle transport, Pierre Habouzit, (Tue Sep 11, 4:23 am)
Re: [PATCH 12/13] Add bundle transport, Johannes Schindelin, (Tue Sep 11, 12:26 pm)
Re: [PATCH 12/13] Add bundle transport, Pierre Habouzit, (Tue Sep 11, 1:30 pm)
Re: [PATCH 12/13] Add bundle transport, Nicolas Pitre, (Tue Sep 11, 2:09 pm)
Re: [PATCH 12/13] Add bundle transport, Johannes Schindelin, (Tue Sep 11, 1:55 pm)
Re: [PATCH 12/13] Add bundle transport, Linus Torvalds, (Tue Sep 11, 1:46 pm)