<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Jun 19, 2013 at 11:59 AM, Rafael Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On 19 June 2013 13:46, Sean Silva <<a href="mailto:silvas@purdue.edu">silvas@purdue.edu</a>> wrote:<br>

> What is up with the naming of this function? Isn't the naming convention for<br>
> the surrounding sys::fs code like_this instead of LikeThis?<br>
> Sorry for the late review.<br>
<br>
</div></div>Good question. I am really uncertain about what the best name style<br>
is. The guideline in<br>
<a href="http://llvm.org/docs/CodingStandards.html#name-types-functions-variables-and-enumerators-properly" target="_blank">http://llvm.org/docs/CodingStandards.html#name-types-functions-variables-and-enumerators-properly</a><br>

is that only STL like classes should have methods_named_like_this, but<br>
quiet a bit of the new path, file system and a bit Object have names<br>
like that :-(<br></blockquote><div><br></div><div style>Regardless, as a function, shouldn't it at least be `getUniqueID` by our conventions (functions start with lowercase)? I think that the snake_case is because the library is modeled after one of the boost libs (which IIRC is slated for inclusion in C++14).</div>
<div style><br></div><div style>-- Sean Silva</div></div></div></div>