Neil Conway <neilc@xxxxxxxxxxx> writes: > Mohan, Ross wrote: >> 2) Not clear (to me, a nonprogrammer) whether this is GNU C, ANSI C, >> "Postgres" C, or what the overall coding protocol is. > Postgres is mostly ANSI C89, with limited use of GNU C extensions (which > should be limited to inside #ifdef __GNUC__ blocks, AFAIK). s/mostly/entirely/ and s/should be/are/, please. If you can identify specific violations of those coding rules, we are all ears. BTW, we really really need some more non-gcc compilers in the buildfarm. I realized this afternoon that CVS tip is breaking the perennial bugaboo that gcc does not complain about static int foo(...); ... int foo(...) { ... } that is omitting "static" from the actual routine definition --- which plenty of other compilers do complain about; so why don't we have a few buildfarm members that will yap loud and long about this? regards, tom lane ---------------------------(end of broadcast)--------------------------- TIP 2: you can get off all lists at once with the unregister command (send "unregister YourEmailAddressHere" to majordomo@xxxxxxxxxxxxxx)