<div dir="ltr">On Wed, May 4, 2016 at 1:43 PM, Aaron Ballman via cfe-commits <span dir="ltr"><<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">jbcoe wrote:<br>
> aaron.ballman wrote:<br>
> > I believe we use "modernize" to really mean "migrate from the old way to the new way", which this definitely fits into since I think the point to this check is to replace bind with better alternatives.<br>
> Would you prefer it to be in `modernize`? I can be easily convinced either way and am happy to move it. If I do move it I might add a script to facilitate doing so.<br>
My preference is for modernize, your preference is for readability, so I say: make @alexfh the tie-breaker! ;-) Alex, what are your thoughts? This seems like a heuristic we may want to state in our documentation to help others decide where to put new checks in the future as well.<br></blockquote><div><br></div><div>FWIW, I'd prefer "modernize", and I'll point out that these waters are muddied by the fact that three of the old ways (boost::bind, std::bind1st, std::bind2nd) all existed prior to C++11, so the fact that <i>one</i> of the old ways (std::bind) was introduced in C++11 doesn't matter so much.</div><div>(I haven't looked, but I'd assume that this clang-tidy check catches all four cases, right?)</div><div><br></div><div>–Arthur</div></div></div></div>