Skip to content
Snippets Groups Projects
Commit 61e98256 authored by Nat Goodspeed's avatar Nat Goodspeed
Browse files

Clarify that items in LLProcess::Params::args are implicitly quoted.

That is, we try to pass through each args entry as a separate child-process
arvg[] entry, whitespace and all.
parent da5d243c
No related branches found
No related tags found
No related merge requests found
...@@ -63,7 +63,16 @@ class LL_COMMON_API LLProcess: public boost::noncopyable ...@@ -63,7 +63,16 @@ class LL_COMMON_API LLProcess: public boost::noncopyable
/// pathname of executable /// pathname of executable
Mandatory<std::string> executable; Mandatory<std::string> executable;
/// zero or more additional command-line arguments /**
* zero or more additional command-line arguments. Arguments are
* passed through as exactly as we can manage, whitespace and all.
* @note On Windows we manage this by implicitly double-quoting each
* argument while assembling the command line. BUT if a given argument
* is already double-quoted, we don't double-quote it again. Try to
* avoid making use of this, though, as on Mac and Linux explicitly
* double-quoted args will be passed to the child process including
* the double quotes.
*/
Multiple<std::string> args; Multiple<std::string> args;
/// current working directory, if need it changed /// current working directory, if need it changed
Optional<std::string> cwd; Optional<std::string> cwd;
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment