[Lldb-commits] [PATCH] D17363: Add SymbolFilePDB with basic support for line tables.

[Greg Clayton via lldb-commits]

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 <jingham at 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 at lists.llvm.org> wrote:
>> 
>> 
>> 
>> On Tue, Mar 1, 2016 at 10:10 AM Greg Clayton <clayborg at 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 at lists.llvm.org
>> http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
>