On 9/08/2016 11:09 a.m., Alex Rousskov wrote:
> On 08/08/2016 08:12 AM, Amos Jeffries wrote:
>> Coverity Scan latest checks are reporting that the ttl member of
>> Ipc::MemMap is being left uninitialized.
>>
>> It sounds like something which would lead to major bugs. Except that it
>> turns out, AFAICS, that this ttl member is never actually being used.
>>
>> Anyone have a reason not to simply drop it from the code?
> 
> There cannot be valid reasons to keep unused code for more than a year.
> Please remove it.
> 
> FWIW, I bet it is just accidental leftovers from some earlier MemMap design.
> 
> Alex.
> 

Thanks. Applied as trunk rev.14781.

Amos

_______________________________________________
squid-dev mailing list
squid-dev@lists.squid-cache.org
http://lists.squid-cache.org/listinfo/squid-dev

Reply via email to