[llvm] r184217 - Add a GetUniqueID that will replace the uniqueID of PathV1.h.

Sean Silva silvas at purdue.edu
Wed Jun 19 14:14:27 PDT 2013


On Wed, Jun 19, 2013 at 1:57 PM, Rafael EspĂ­ndola <
rafael.espindola at gmail.com> wrote:

> > Regardless, as a function, shouldn't it at least be `getUniqueID` by our
> > conventions (functions start with lowercase)?
>
> Yes, my mistake. I will rename it once Aaron's patch is in.
>
> > 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).
>
> Which part? Both Path.h and FileSysetem.h? I guess once the V1 to V2
> transition is complete the next cleanup in this area is then to split
> the headers along the lines of what is slatted for c++14.
>
>
My knowledge mostly derives from the header comment in Path.h: <
http://llvm.org/docs/doxygen/html/Path_8h_source.html#l00010>

"It is designed after TR2/boost filesystem (v3)"
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130619/d34e86e2/attachment.html>


More information about the llvm-commits mailing list