<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Jun 19, 2013 at 1:57 PM, 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:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div class="im">> Regardless, as a function, shouldn't it at least be `getUniqueID` by our<br>

> conventions (functions start with lowercase)?<br>
<br>
</div>Yes, my mistake. I will rename it once Aaron's patch is in.<br>
<div class="im"><br>
> I think that the snake_case is<br>
> because the library is modeled after one of the boost libs (which IIRC is<br>
> slated for inclusion in C++14).<br>
<br>
</div>Which part? Both Path.h and FileSysetem.h? I guess once the V1 to V2<br>
transition is complete the next cleanup in this area is then to split<br>
the headers along the lines of what is slatted for c++14.<br><div class=""><div class="h5"><br></div></div></blockquote><div><br></div><div style>My knowledge mostly derives from the header comment in Path.h: <<a href="http://llvm.org/docs/doxygen/html/Path_8h_source.html#l00010">http://llvm.org/docs/doxygen/html/Path_8h_source.html#l00010</a>></div>
<br>"It is designed after TR2/boost filesystem (v3)"</div><br></div></div>