<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Jul 21, 2014 at 10:31 AM, Duncan P. N. Exon Smith <span dir="ltr"><<a href="mailto:dexonsmith@apple.com" target="_blank">dexonsmith@apple.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div id=":6mu" class="a3s" style="overflow:hidden">BTW, I don't see the value in these mass refactorings to use C++11<br>

range-based for loops.  While adding the API is obviously good, changing<br>
everything to use it means that these mistakes are going to creep in,<br>
and it's hard enough to review these that they won't get many eyes.<br>
<br>
I think it would be better if the next person to touch the code cleaned<br>
it up (where appropriate).  If they're touching the code anyway, they'll<br>
have the time to understand the logic and be careful about it.</div></blockquote></div><br>FWIW, I *strongly* agree.</div><div class="gmail_extra"><br></div><div class="gmail_extra">There was one place where I updated the users of the API and that was because *all of them had to change anyways*. And it was miserable, horrid, and required a ton of testing and review to shake out the bugs.</div>
<div class="gmail_extra"><br></div><div class="gmail_extra">Please, don't "fix" code in ways that can introduce subtle bugs unless you're working on that code anyways, and do so in small chunks that are easier to test, revert, and isolate when bugs sneak in.</div>
</div>