Thread (1 message) 1 message, 1 author, 2016-06-15

Re: [PATCH] Avoid C++ comments, use C comments instead

From: Junio C Hamano <hidden>
Date: 2016-06-15 22:42:33

Shawn Pearce [off-list ref] writes:
Olivier Galibert [off-list ref] wrote:
quoted
Maybe git can be perfectly happy with c89.  I don't know.  I know the
linux kernel requires c99, mostly for the struct initializers.  My
point was that staying at the c89 level has a maintainance cost, and a
cost/benefit analysis should be done to decide whether it is a good
idea.
I am generally in favor of the effect C99 struct and array
initializers have on the readability, but we also need to
balance that with the reality.  The patch is only about a few
structs and one array if I recall correctly isn't it?

What Olivier says is perfectly correct, and after "cost/benefit
analysis", I would have to say avoiding some C99 is fine if that
makes people's life on some major non-Linux platforms easier.

C99 clean-up is already in the "master", so please do not waste
more bandwidth nor time on this issue, but instead spend time
elsewhere to make our system better for more people ;-).

Thanks.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help