<div dir="ltr"><div>I prefer east/west, but I think there's a terminology that is uncontroversial (left/right) and one that is controversial. It's also clear to me that it's better to have only one set of terms (aliases are only fine for backwards compatibility). Left/Right won, I think.</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, May 26, 2020 at 1:55 PM Sam McCall via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">sammccall added a comment.<br>
<br>
Thanks @MyDeveloperDay for hammering on on these bugs and @steveire for finding them! There's still obviously some risk here but as long as this is opt-in for a major release or two (i.e. not turned on in built-in styles, any remaining bugs should get flushed out.<br>
<br>
Regarding option naming, I did think East/West (only) was the way to go but have reluctantly changed my mind after rereading the discussion. My conclusions were:<br>
<br>
- C++ people who have read discussions on this from 2018 on are likely familiar with "east const" terminology<br>
- there are people who care about style who aren't familiar with it and wouldn't be happy to have to learn/adopt it (this was the main surprise for me)<br>
- 5 years from now, this naming may have spread to ~everyone, may remain partially-adopted, or possibly even die out as a fad<br>
- for people familiar with the terminology: "ConstStyle: East" is clearer than "ConstStyle: Right" (less ambiguous)<br>
- for people unfamiliar with the terminology, the opposite is certainly true<br>
- asymmetry 1: it's probably harder to work out east=right than to to work out that "ConstStyle: right" means const is written on the right of the type<br>
- asymmetry 2: the new terminology is objectively better: terse, memorable, doesn't collide with other terms. Some dislike it, which is true of any distinctive name (I hate "namespace", for instance).<br>
- there's clearly a cultural tension between reading like a meme-infested subreddit and like an IBM technical manual :-)<br>
<br>
It's a tough call, but I'd propose a compromise: make Left/Right canonical as it's more accessible (don't have to learn new things) and in case East/West dies out.<br>
But to have East/West as aliases: to let the community decide over time, and because I don't think we should be in the business of hindering adoption of better names.<br>
<br>
("reluctantly" changed my mind because I do think east/west are better names. But meeting users where they are is important too - otherwise we'd just hardcode the One True Style :-))<br>
<br>
<br>
CHANGES SINCE LAST ACTION<br>
  <a href="https://reviews.llvm.org/D69764/new/" rel="noreferrer" target="_blank">https://reviews.llvm.org/D69764/new/</a><br>
<br>
<a href="https://reviews.llvm.org/D69764" rel="noreferrer" target="_blank">https://reviews.llvm.org/D69764</a><br>
<br>
<br>
<br>
</blockquote></div>