Re: [msysGit] Re: [PATCH 02/11] Fix declare variable at mid of function

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Johannes Schindelin said the following on 17.08.2009 21:36:
Hi,

On Mon, 17 Aug 2009, Reece Dunn wrote:

2009/8/17 Johannes Schindelin <Johannes.Schindelin@xxxxxx>:

On Tue, 18 Aug 2009, Frank Li wrote:

Some compiler such as MSVC can't support declear variable at mid of funtion at c file.
Please wrap your commit messages after 76 characters.

Signed-off-by: Frank Li <lznuaa@xxxxxxxxx>
---
How about this instead?

       Avoid declaration after instruction

       Microsoft Visual C++ does not understand this C99 style.

?

The patch itself is good.
Shouldn't GCC be changed to use -std=c89 as well to pick up errors for compilers that don't support c99 (like the Microsoft Visual C++ C compiler)?

Hmm. I played with the thought of adding -Werror -Wno-pointer-to-int-cast -Wold-style-definition -Wdeclaration-after-statement like Junio described in one of his mails for MinGW (as we _know_ what compiler we have there).

Dunno.

IMO it would be a good change. We should  not have any of those anyways..

--
.marius
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]