[Lldb-commits] [PATCH] Invalidate process uid/gid command options on Windows

[Greg Clayton]

Looks good and I see "CommandInterpreter &" was added to IsValid() as I commented on in the previous patch comments.

Greg


> On Jul 3, 2014, at 2:24 AM, Zachary Turner <zturner at google.com> wrote:
> 
> This patch implements the core logic to invalidate command options based on the interpreter state, and uses this to invalidate uid/gid related options of the "platform process list" command on Windows.  
> 
> When an invalid option is specified for a command, an error message is printed that explains the reason that the option is invalid, and furthermore when the help of a command is printed, options which have runtime validity conditions display an abbreviated-condition string in square braces prior to the long description of the option.
> 
> http://reviews.llvm.org/D4373
> 
> Files:
>  include/lldb/Host/OptionParser.h
>  include/lldb/Interpreter/CommandInterpreter.h
>  include/lldb/Interpreter/CommandOptionValidators.h
>  include/lldb/Interpreter/Options.h
>  include/lldb/Target/TargetList.h
>  include/lldb/lldb-private-types.h
>  source/Commands/CommandObjectPlatform.cpp
>  source/Interpreter/Args.cpp
>  source/Interpreter/CMakeLists.txt
>  source/Interpreter/CommandOptionValidators.cpp
>  source/Interpreter/Options.cpp
> <D4373.11047.patch>_______________________________________________
> lldb-commits mailing list
> lldb-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits