diff options
author | Marc G. Fournier <scrappy@hub.org> | 1997-06-06 00:37:23 +0000 |
---|---|---|
committer | Marc G. Fournier <scrappy@hub.org> | 1997-06-06 00:37:23 +0000 |
commit | f50b103ce1b0aca3f63d47865dcf6f89ded158e5 (patch) | |
tree | f5fa4bf9ed5e273eacea55711a04905900916aa7 /src/tutorial/C-code/funcs.c | |
parent | a668b7ac2d95bbac8107c4622de2ea4962909c04 (diff) | |
download | postgresql-f50b103ce1b0aca3f63d47865dcf6f89ded158e5.tar.gz postgresql-f50b103ce1b0aca3f63d47865dcf6f89ded158e5.zip |
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.
Diffstat (limited to 'src/tutorial/C-code/funcs.c')
0 files changed, 0 insertions, 0 deletions