Thread (24 messages) 24 messages, 6 authors, 2016-06-15

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

flat view

From: Nicolas Pitre <hidden>
Date: 2016-06-15 22:46:54

On Fri, 5 Jun 2009, Brandon Casey wrote:
From: Brandon Casey <redacted>

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 <redacted>
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
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help