<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, Oct 6, 2014 at 5:00 PM, Nick Kledzik <span dir="ltr"><<a href="mailto:kledzik@apple.com" target="_blank">kledzik@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 style="word-wrap:break-word"><span class="">On Oct 6, 2014, at 3:44 PM, Rui Ueyama <<a href="mailto:ruiu@google.com" target="_blank">ruiu@google.com</a>> wrote:<br></span><div><span class=""><blockquote type="cite"><div dir="ltr">Looks like most people in this thread support using LLVM style in LLD. I also had an offline discussion and many people wanted to have one coding style in all LLVM projects. So I'm convinced that we should do that.<div><br></div><div>I'm going to create a patch to rename all variables if no one objects.</div></div></blockquote></span><div>I object!</div></div></div></blockquote><div><br></div><div>I'd like to hear the reason. :)</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word"><div><span class="HOEnZb"><font color="#888888"><div><br></div><div>-Nick</div><br></font></span><blockquote type="cite"><span class=""><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Oct 6, 2014 at 3:10 PM, Bruce Hoult <span dir="ltr"><<a href="mailto:bruce@hoult.org" target="_blank">bruce@hoult.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><span>On Tue, Oct 7, 2014 at 8:54 AM, Rui Ueyama <span dir="ltr"><<a href="mailto:ruiu@google.com" target="_blank">ruiu@google.com</a>></span> wrote:<br></span><div class="gmail_extra"><div class="gmail_quote"><span><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">would touch all the lines. Diff is not powerful enough to trace the history beyond variable renaming. svn blame would become useless.</div></blockquote><div><br></div></span><div>"useless" is much too strong a description.</div><div><br></div><div>If svn blame points to the reformat commit, then simply re-run the blame on the commit before the reformat.</div><div><br></div><div>If total reformats are frequent then this is impractical. If they happen once in five years then it's not really a big deal. Wrapper scripts can even be pretty easily written to do this automatically.</div><div><br></div></div></div></div>
</blockquote></div><br></div></span><span class="">
_______________________________________________<br>LLVM Developers mailing list<br><a href="mailto:LLVMdev@cs.uiuc.edu" target="_blank">LLVMdev@cs.uiuc.edu</a>         <a href="http://llvm.cs.uiuc.edu" target="_blank">http://llvm.cs.uiuc.edu</a><br><a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev</a><br></span></blockquote></div><br></div></blockquote></div><br></div></div>