diff options
| author | Junio C Hamano <gitster@pobox.com> | 2017-02-13 11:14:24 -0800 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2017-02-18 12:50:29 -0800 |
| commit | 70fc91272ca30a985567bdc85d74dd3afafda1f1 (patch) | |
| tree | 58658c36b255336f9a4f46ca1b7130c4bd435064 /commit.c | |
| parent | 9889c96293f80f58a6f96d1d39709fbb37bb9787 (diff) | |
| download | git-pb/bisect.tar.gz | |
fixup! bisect--helper: `bisect_next_check` & bisect_voc shell function in Cpb/bisect
René Scharfe <l.s.r@web.de> writes:
> Initializing to NULL is still the correct thing to do, of course --
> together with removing the conditionals (or at least the negations).
So, let's give Pranit a concrete "here is what we want to see
squashed in", while you guys discuss peculiarity with various
platforms and their system headers, which admittedly is a more
interesting tangent ;-)
There are early returns with "goto finish" even before _syn
variables are first assigned to, so they would need to be
initialized to NULL. The other two get their initial values
right at the beginning, so they are OK.
builtin/bisect--helper.c | 14 +++++---------
1 file changed, 5 insertions(+), 9 deletions(-)
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'commit.c')
0 files changed, 0 insertions, 0 deletions
