Re: [HACKERS] Patch to log usage of temporary files

Bill Moran wrote:
> In response to "Andrew Dunstan" <andrew@dunslane.net>:
>
>
>> Bill Moran wrote:
>>
>>> Andrew Dunstan <andrew@dunslane.net> wrote:
>>>
>>>> Bill Moran wrote:
>>>>
>>>>> +         if (trace_temp_files != -1)
>>>>>
>>>>>
>>>> Might be more robust to say
>>>>
>>>>     if (trace_temp_files >= 0)
>>>>
>>> Because it would allow for the easy addition of more negative numbers
>>> with magic value?
>>>
>> because ISTM any negative number here should mean no action is to be
>> taken. Otherwise how else is it different from 0?
>>
>
> ??
>
> I specified in the GUC config that minimum allowable value is -1.
>
>


OK, missed that, sorry. Please return to normal viewing ...

cheers

andrew