OSDN Git Service

From: "D'Arcy J.M. Cain" <darcy@druid.net>
authorMarc G. Fournier <scrappy@hub.org>
Fri, 6 Jun 1997 00:37:23 +0000 (00:37 +0000)
committerMarc G. Fournier <scrappy@hub.org>
Fri, 6 Jun 1997 00:37:23 +0000 (00:37 +0000)
commitf50b103ce1b0aca3f63d47865dcf6f89ded158e5
treef5fa4bf9ed5e273eacea55711a04905900916aa7
parenta668b7ac2d95bbac8107c4622de2ea4962909c04
From: "D'Arcy J.M. Cain" <darcy@druid.net>
Subject: [HACKERS] backend/optimizer/geqo/geqo_erx.c

I sent these changes in with a bunch of others.  Some were folded in but
others, like these, were not.  I am not sure why so I am resending this
to the developers list by itself for discussion.

The readon why I suggest these changes is that the compiler can't tell
that minimum_count is initialized before it is used.  The tests that I
add in here will cause an immediate error if it doesn't.  As the comments
below suggest, if it is 100% guaranteed that the variable will always
be initialized then how this is so should be commented here.  I don't
know how much strain the actual test puts on the performance but if it
isn't too much then maybe leave it in for absolute safety anyway.

There are also a few returns just to stop warnings.
src/backend/optimizer/geqo/geqo_erx.c