On 7/04/2016 6:10 p.m., Alex Rousskov wrote:
> Hello,
> 
>     The attached patch removes broken and, AFAICT, unused "partial copy"
> ServerOptions copy constructor.
> 
> I am not intimate with the affected code, so more work may be needed
> around this change: If the partial copy constructor was meant to be
> abused for something, then a new approach should be found for that use
> case -- an approach that does not violate the standard copy constructor
> post-conditions.

A (full) copy-constructor is needed on some systems for the clear()
method implementation to build.

Amos

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

Reply via email to