<div dir="ltr"><div dir="ltr">On Fri, Feb 22, 2019 at 9:40 AM David Greene via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org">llvm-dev@lists.llvm.org</a>> wrote:<br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">> So:<br>
><br>
> - Local variables and formal parameters should be lower_case, with<br>
>   one exception: Variables/parameters that have lambda/function<br>
>   type should follow the function-name spelling rules.<br>
<br>
"lower_case" is a pretty drastic change from CamelCase and camelCase.<br>
So far the only argument for it I've seen is, "lldb uses it all over the<br>
place."  Having one subproject drive the standard for everything else<br>
seems backward.  It's certainly possible I missed other opinions on this<br>
though.<br></blockquote><div><br></div><div>This is not how I described my preference for it. I prefer it generally and find it substantially easier to read code using it.</div><div><br></div><div>That said, I think the fact that LLDB uses it does contribute two motivations:</div><div>1) It does show some subset of the community is familiar with it and has found it to be a reasonably good convention for their usage.</div><div>2) It removes some cost of transition.</div><div><br></div><div>I wouldn't make the decision *because* of this, but I also don't think we should ignore these points.</div></div></div>