labath added a comment.

Actually, my plan was to follow this up with a patch which removes usages of 
that enumeration from the internal api altogether (basically do a 
`s/GetLLDBPath(ePathTypeXXX/GetXXXPath`). That should make the internal api 
nicer, and provide a clean separation between the internal and public api 
(thereby taking pressure off of adding new enums to the public api just because 
we have a new kind of an internal path).

I was considering in which order to submit the patches. In the end I chose this 
one because it will make the second patch slightly smaller, but I can do it in 
the other order too...


https://reviews.llvm.org/D48215



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to