On 03/12/2017 07:45 AM, khaled belhout wrote: > this patch modernize for loops using c++11 Range-based for loop
Please use "const auto" where possible and avoid using "i" for naming the current iteration object. I am curious why did you decide to change all these loops? How did you select the loops to change? Normally, we avoid wide-spread polishing touches to minimize the price developers working with older code have to pay when porting back various bug fixes and features. I am trying to decide whether the advantages of changing these loops outweigh those costs in this case. Thank you, Alex. _______________________________________________ squid-dev mailing list [email protected] http://lists.squid-cache.org/listinfo/squid-dev
