<html><head><meta http-equiv="Content-Type" content="text/html charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;">On Oct 6, 2014, at 3:44 PM, Rui Ueyama <<a href="mailto:ruiu@google.com">ruiu@google.com</a>> wrote:<br><div><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><div>I object!</div><div><br></div><div>-Nick</div><br><blockquote type="cite"><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 class="">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 class=""><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>
_______________________________________________<br>LLVM Developers mailing list<br><a href="mailto:LLVMdev@cs.uiuc.edu">LLVMdev@cs.uiuc.edu</a> <a href="http://llvm.cs.uiuc.edu">http://llvm.cs.uiuc.edu</a><br><a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev">http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev</a><br></blockquote></div><br></body></html>