[Lldb-commits] [PATCH] D29496: Push down more common code into PlatformPOSIX

2017-02-03 Thread Pavel Labath via Phabricator via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL294019: Push down more common code into PlatformPOSIX (authored by labath). Changed prior to commit: https://reviews.llvm.org/D29496?vs=86975&id=86982#toc Repository: rL LLVM https://reviews.llvm.or

[Lldb-commits] [PATCH] D29496: Push down more common code into PlatformPOSIX

2017-02-03 Thread Greg Clayton via Phabricator via lldb-commits
clayborg added a comment. indeed very nice https://reviews.llvm.org/D29496 ___ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [PATCH] D29496: Push down more common code into PlatformPOSIX

2017-02-03 Thread Kamil Rytarowski via Phabricator via lldb-commits
krytarowski accepted this revision. krytarowski added a comment. Thank you for this patch. https://reviews.llvm.org/D29496 ___ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [PATCH] D29496: Push down more common code into PlatformPOSIX

2017-02-03 Thread Pavel Labath via Phabricator via lldb-commits
labath created this revision. - GetFileWithUUID: All platforms except PlatformDarwin had this. However, I see no reason why this code would not apply there as well. - GetProcessInfo, FindProcesses: The implementation was the same in all classes. - GetFullNameForDylib: This code should apply to