[Lldb-commits] [PATCH] D56196: ProcessLaunchInfo: Remove Target reference

Pavel Labath via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Wed Jan 2 01:43:36 PST 2019

labath created this revision.
labath added reviewers: zturner, jingham, teemperor.

The target was being used in FinalizeFileActions to provide default
values for stdin/out/err. Also, most of the logic of this function was
very specific to how the lldb's Target class wants to launch processes,
so I, move it to Target::FinalizeFileActions, inverting the dependency.
The only piece of logic that was useful elsewhere (lldb-server) was the
part which sets up a pty and relevant file actions. I've kept this part
as ProcessLaunchInfo::SetUpPtyRedirection.

This makes ProcessLaunchInfo independent of any high-level lldb constructs.



-------------- next part --------------
A non-text attachment was scrubbed...
Name: D56196.179824.patch
Type: text/x-patch
Size: 13694 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20190102/e1886382/attachment-0001.bin>

More information about the lldb-commits mailing list