<div dir="auto">Also, we only use auto if the type of the variable is clear. Changes like<div dir="auto"><br></div><div dir="auto"><span style="font-family:sans-serif;font-size:12.8px">-  for (ModuleFile &I : llvm::reverse(ModuleMgr)) {</span><br style="font-family:sans-serif;font-size:12.8px"><span style="font-family:sans-serif;font-size:12.8px">+  for (auto &I : llvm::reverse(ModuleMgr)) {</span><br></div><div dir="auto"><span style="font-family:sans-serif;font-size:12.8px"><br></span></div><div dir="auto">are not desired<font face="sans-serif"><span style="font-size:12.8px">.</span></font></div></div><br><div class="gmail_quote"><div dir="ltr">On Sat, Apr 14, 2018, 11:09 AM Malcolm Parsons via cfe-commits <<a href="mailto:cfe-commits@lists.llvm.org">cfe-commits@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="auto"><div><div class="gmail_quote"><div dir="ltr">On Sat, 14 Apr 2018, 14:16 Kim Gräsman, <<a href="mailto:kim.grasman@gmail.com" target="_blank" rel="noreferrer">kim.grasman@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="auto">That would be a nice outcome of all the "run-tools-on-llvm" changes if any problems were filed as bugs on the tools. We have a number of them filed on iwyu, and they make for nice, concrete bugs to troubleshoot even if we don't always know how to fix them. <div dir="auto"><br></div><div dir="auto">For this specific clang-tidy issue, do you have any ideas for how to tell this loop apart from any other? I'm guessing the container is modified while iterating... Or do you mean skip all non-iterator loops?</div></div></blockquote></div></div><div dir="auto"><br></div><div dir="auto"><div dir="auto">Non-iterator, mutable container, size checked each iteration.</div><div dir="auto"><br></div><div dir="auto">Clang-tidy could suggest modernisation, but not automatically fix.</div><div dir="auto" style="font-family:sans-serif;font-size:12.8px"><br></div><div dir="auto" style="font-family:sans-serif;font-size:12.8px"><div data-smartmail="gmail_signature" style="font-size:large" dir="auto">-- <br>Malcolm Parsons</div></div></div></div>
_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@lists.llvm.org" target="_blank" rel="noreferrer">cfe-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits" rel="noreferrer noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits</a><br>
</blockquote></div>