Re: rmtree cleanup

Tom Lane wrote:

>Andrew Dunstan <andrew@dunslane.net> writes:
>
>
>>The attached patch cleans up src/port/dirmod.c::rmtree() a bit. It moves
>>the filename finding portion into its own function, and in that function
>>only scans the directory once instead of twice.
>>
>>
>
>Applied, along with some further hacking to reduce the #ifdef clutter
>by providing palloc substitute routines.
>

Nice. It's actually readable now :-)

>I checked it still works on
>Unix, would you check I didn't break the Windows cases?
>
>
>
>

Yes, it works. Thanks.

andrew