labath added a comment.

Seems like a reasonable thing to do. However, we should first add a test for 
the new functionality.


================
Comment at: source/Commands/CommandObjectPlatform.cpp:2041
@@ -1969,3 +2040,3 @@
                                 "A set of commands to query, launch and attach 
to platform processes",
                                 "platform process [attach|launch|list] ...")
     {
----------------
Please update this as well.
With this many suboptions, we can go for something like
`platform process <command> [<command-options>]`, as other commands do.


http://reviews.llvm.org/D14952



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

Reply via email to