You don't add plug-in specify unique_ptr and shared_ptr definitions. You would 
put these in SymbolFilePDB.h only.


> On Mar 1, 2016, at 10:51 AM, Jim Ingham <jing...@apple.com> wrote:
> 
> There's an lldb-private-forward.h that you can use for this purpose if you 
> wish.
> 
> Jim
> 
>> On Mar 1, 2016, at 10:32 AM, Zachary Turner via lldb-commits 
>> <lldb-commits@lists.llvm.org> wrote:
>> 
>> 
>> 
>> On Tue, Mar 1, 2016 at 10:10 AM Greg Clayton <clayb...@gmail.com> wrote:
>> clayborg requested changes to this revision.
>> clayborg added a comment.
>> This revision now requires changes to proceed.
>> 
>> One general comment is the use of "auto". Although it makes the code 
>> shorter, it does make it quite a bit less readable. I will leave the 
>> decision to you since this is your code, but in general I think this is 
>> where auto is less than it is cracked up to be.
>> I agree with you in principle, but I used it here because these types are 
>> really ugly because they're unique pointers of types whose names are already 
>> long.  I could typedef them to UP or something, but they wouldn't be able to 
>> go in lldb-forward.h because they shouldnt be publicly exposed.  I'll think 
>> about it some more and decide what to do.
>> 
>> Have to run right now, will respond to the rest of your comments later.
>> 
>> _______________________________________________
>> lldb-commits mailing list
>> lldb-commits@lists.llvm.org
>> http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
> 

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

Reply via email to