janitor: use NULL and not 0 for pointers.

MadCoder · gitster · commit 67da52bedc57 · 2009-07-22T21:57:41.000-07:00
Brought to you thanks to coccinelle:

    ---8&lt;----
    @@
    expression *E;
    @@
    (
      E ==
    - 0
    + NULL
    |
      E !=
    - 0
    + NULL
    |
      E =
    - 0
    + NULL
    )

    @@
    identifier f;
    type T;
    @@
    T *f(...) {
    &lt;...
    - return 0;
    + return NULL;
    ...&gt;
    }
    ---&gt;8----

There are a lot more hits in compat/nedmallox and compat/regex but these
are borrowed code we rather do not want to maintain our own forks for,
and this patch refrains from touching them.

Signed-off-by: Pierre Habouzit &lt;madcoder@debian.org&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
diff --git a/graph.c b/graph.c
@@ -893,7 +893,7 @@ static struct column *find_new_column_by_commit(struct git_graph *graph,
 		if (graph->new_columns[i].commit == commit)
 			return &graph->new_columns[i];
 	}
-	return 0;
+	return NULL;
 }
 
 static void graph_output_post_merge_line(struct git_graph *graph, struct strbuf *sb)

Original file line number	Diff line number	Diff line change
`@@ -893,7 +893,7 @@ static struct column find_new_column_by_commit(struct git_graph graph,`
`893`	`893`	`if (graph->new_columns[i].commit == commit)`
`894`	`894`	`return &graph->new_columns[i];`
`895`	`895`	`}`
`896`		`- return 0;`
	`896`	`+ return NULL;`
`897`	`897`	`}`
`898`	`898`
`899`	`899`	`static void graph_output_post_merge_line(struct git_graph graph, struct strbuf sb)`