<div dir="ltr">I want to reiterate the benefit that underscore_names would bring.  To be clear it's not my favorite style, but it does have a very concrete advantage which is that we have a very large subproject already using it.  it doesn't make sense to do a purely aesthetic move that not everyone is going to agree on anyway, when we could do one with actual tangible value.</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Feb 13, 2019 at 8:52 AM <<a href="mailto:paul.robinson@sony.com">paul.robinson@sony.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Chandler wrote:<br>
<br>
> FWIW, I'm pretty strongly opposed to humbleCamelCase. We already use that<br>
> style so something else. <br>
<br>
Presumably you are equally opposed to RegularCamelCase, because we already<br>
use *that* style for something else.<br>
<br>
But really, objecting on the grounds that a given style is already used for<br>
function names is really a very weak argument.  IME function names are <br>
*incredibly* *hard* to confuse with anything else, because they *always* have <br>
surrounding syntactic context. Given `TheStuff->fooBar().getThingy()` is it<br>
even conceivable that you might not instantly get that fooBar and getThingy<br>
are methods?  Therefore, using the same convention for some other kind of<br>
name is Not Confusing.<br>
<br>
OTOH, `TheStuff` comes out of nowhere with no clues to its origin, and *that*<br>
is a barrier to code-reading IME.  Even renaming it to `stuff` would help<br>
approximately zero percent. Parameter? Local? Class member? Global?  LLVM has <br>
incredibly few globals for other reasons, but using the same convention for <br>
locals and class members is a real problem for code-reading, especially code <br>
operating in methods for classes you're not super familiar with.<br>
<br>
I acknowledge that the current RFC doesn't propose a member naming convention<br>
different from other variables, but IMO it really ought to.  *That* is the<br>
distinction that would really help in reading unfamiliar code.<br>
--paulr<br>
</blockquote></div>