Andrew Dunstan <[EMAIL PROTECTED]> writes:
> Revised patch attached, doing just this. I will apply it soon unless 
> there are objections.

Probably a good idea to check defined(HAVE_GETRLIMIT) && defined(RLIMIT_CORE),
rather than naively assuming every getrlimit implementation supports
that particular setting.  Also, should the -c option exist but just not
do anything if the platform doesn't support it?  As is, you're making it
impossible to just specify -c without worrying if it does anything.

The documentation fails to list the long form of the switch
(--corefiles, which should probably really be --core-files for consistency).
There's a typo in this message, too:

+                               _("%s: cannot set core size,: disallowed by 
hard limit.\n"), 

                        regards, tom lane

---------------------------(end of broadcast)---------------------------
TIP 6: explain analyze is your friend

Reply via email to