Re: [PATCH v2 3/8] diff-delta.c: "delta.h" is not a required include

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

 



On Fri, 5 Jun 2009, Brandon Casey wrote:

> From: Brandon Casey <drafnel@xxxxxxxxx>
> 
> When compiling diff-delta.c with the SUNWspro C99 compiler, it complains
> 
>     "diff-delta.c", line 314: identifier redeclared: create_delta
> 
> There is nothing in "delta.h" that is required by diff-delta.c, so don't
> include it.
> 
> Signed-off-by: Brandon Casey <drafnel@xxxxxxxxx>

NAK.

This is common practice to include the header file declaring function 
prototypes into the file defining the actual function so to make sure 
the declaration matches with the definition.  Deleting that include is 
actively ignoring a problem instead of fixing the cause of it.


Nicolas
--
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]