> I'm going to push back on that; the administrator doesn't really have
> any need to get a core when, for example, append_domain doesn't start
> with .'.
>
> Squid.conf is bloated as it is; if there are cases where a core could
> be conceivably useful, they should be converted to fatal_dump. From
> what I've seen they'll be a small minority at best...
>
That I agree with. Grep the code for all fatals and see what falls out.
I think you will find it's only the config parser that can get away with
non-core fatals.
Amos
>
>
>
> On 19/05/2009, at 2:25 PM, Adrian Chadd wrote:
>
>> just make that behaviour configurable?
>>
>> core_on_fatal {on|off}
>>
>>
>>
>> Adrian
>>
>> 2009/5/19 Mark Nottingham <mnot_at_yahoo-inc.com>:
>>> tools.c:fatal() dumps core because it calls abort.
>>>
>>> Considering that the core can be quite large (esp. on a 64bit
>>> system), and
>>> that there's fatal_dump() as well if you really want one, can we
>>> just make
>>> fatal() exit(1) instead of abort()ing?
>>>
>>> Cheers,
>>>
>>> --
>>> Mark Nottingham mnot_at_yahoo-inc.com
>>>
>>>
>>>
>
> --
> Mark Nottingham mnot_at_yahoo-inc.com
>
>
>
Received on Wed May 20 2009 - 01:24:46 MDT
This archive was generated by hypermail 2.2.0 : Wed May 20 2009 - 12:00:03 MDT