The final part of this applied as rev.14218. Amos
On 1/08/2015 2:50 p.m., Amos Jeffries wrote: > On 1/08/2015 12:40 p.m., Alex Rousskov wrote: >> Hello, >> >> The attached patch fixes trunk handling of large transactions broken >> by r14093 (Replace Packer object API with Packable API). I only had a >> few minutes to test this so further work in this direction may be necessary. >> >> The patch also adds a missing virtual destructor to the new Packable API. >> > > Must be right. I also came up with exactly identical set of flush() > calls independently based on what you said in IRC. :-) > > Sorry, I applied that already before seeing this mail. > > Not sure about the destructor part. IIRC there were some issues when I > tried to add it during the Packable development. Though that may have > been with the SBuf bit of the project that is not in trunk. > Will test it out after the releases and apply if working. > > Thanks. > > Amos > > _______________________________________________ > squid-dev mailing list > [email protected] > http://lists.squid-cache.org/listinfo/squid-dev > _______________________________________________ squid-dev mailing list [email protected] http://lists.squid-cache.org/listinfo/squid-dev
