[reposting with gzipped patch] Here is a patch further cleaning up dummy process startup and the bootstrap code itself a little.
I've tested it manually giving erroneous input and it behaves in the same ways as the original. Of course, it still manages to bootstrap normally and passes regression tests. This patches removes a BOOTSTRAP_INCLUDE symbol from tcopprot.h, since it seems to be useless. It is about trying to avoid getting prototypes for functions not needed in bootstrap, presumably because at some point the bootstrap.c file did not have enough includes to be able to compile those definitions cleanly, or maybe because it had conflicting definitions. But it now works without that. I also took the liberty of renaming "xlog operation" into "dummy process type", since that's what really the things are. I assume it was named "xlog operation" back when the xlog code was written with the idea of extending it into different xlog ops. But we haven't had any. I think this is as far as I'll go with cleaning up this code in this round. Task for some other janitor ... -- Alvaro Herrera http://www.CommandPrompt.com/ The PostgreSQL Company - Command Prompt, Inc.
bootstrap-cleanup.patch.gz
Description: Binary data
---------------------------(end of broadcast)--------------------------- TIP 9: In versions below 8.0, the planner will ignore your desire to choose an index scan if your joining column's datatypes do not match