On Thu, Sep 9, 2010 at 20:23, Tom Lane <tgl@sss.pgh.pa.us> wrote:
> Magnus Hagander <magnus@hagander.net> writes:
>> On Thu, Sep 9, 2010 at 19:48, Tom Lane <tgl@sss.pgh.pa.us> wrote:
>>> On balance I think I'd suggest an #ifdef WIN32 in CleanupBackend that
>>> made it accept 128 as a "normal exit" case.
>
>> Seems reasonable. I'll whack it around for that - see attached.
>
> Hm, still doesn't log, which I think it should, even for testing
> purposes (how will you know the case occurred?). Maybe like this:
Agreed, that's better.
>> Dave has a reasonably reproducible test environment. Unforunately it's
>> on 8.3, so this patch will be completely unsafe there (it doesn't have
>> the deadman switch). But hopefully it can be used to see it fixes this
>> problem (while introducing others)h
>
> Sounds like a plan.
Patch is with dave for testing now :-)
--
Magnus Hagander
Me: http://www.hagander.net/
Work: http://www.redpill-linpro.com/