[Lldb-commits] [PATCH] Invalidate process uid/gid command options on Windows
zturner at google.com
Thu Jul 3 22:00:08 PDT 2014
So, with an ExecutionContext, everything is null if there's no target. For
example, I might not be attached to anything, but trying to list processes
on the host in order to know what process to attach to.
How to cleanly support all of these use cases?
On Thu, Jul 3, 2014 at 11:14 AM, <jingham at apple.com> wrote:
> Again, the validator should take an execution context not an interpreter.
> Some uses of commands, particularly HandleCommands used by breakpoint
> callbacks and the like, don't use the Command Interpreter's execution
> context, rather they are passed in the appropriate environment.
> > 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
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the lldb-commits