luo980/uvw
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Remove always-added stdin in uvw::ProcessHandle (#129)

Browse Source
This commit is contained in:
zxtwonder 2018-10-04 21:58:07 +08:00 committed by Michele Caini
parent 3523ec5439
commit c4864f1a5b
1 changed files with 13 additions and 28 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
src/uvw/process.hpp
View File

@ -73,11 +73,8 @@ public:
using StdIO = details::UVStdIOFlags; using StdIO = details::UVStdIOFlags;
ProcessHandle(ConstructorAccess ca, std::shared_ptr<Loop> ref) ProcessHandle(ConstructorAccess ca, std::shared_ptr<Loop> ref)
: Handle{ca, std::move(ref)}, poFdStdio{1} : Handle{ca, std::move(ref)}
{ {}
// stdin container default initialization
poFdStdio[0].flags = static_cast<uv_stdio_flags>(StdIO::IGNORE_STREAM);
}
/** /**
* @brief Disables inheritance for file descriptors/handles. * @brief Disables inheritance for file descriptors/handles.
@ -135,7 +132,6 @@ public:
uv_process_options_t po; uv_process_options_t po;
po.exit_cb = &exitCallback; po.exit_cb = &exitCallback;
po.file = file; po.file = file;
po.args = args; po.args = args;
po.env = env; po.env = env;
@ -144,13 +140,6 @@ public:
po.uid = poUid; po.uid = poUid;
po.gid = poGid; po.gid = poGid;
/**
* See the constructor, poFdStdio[0] is stdin. It must be poStdio[0] by
* convention. From the official documentation:
*
* > The convention is that stdio[0] points to stdin, fd 1 is used
* > for stdout, and fd 2 is stderr.
*/
std::vector<uv_stdio_container_t> poStdio; std::vector<uv_stdio_container_t> poStdio;
poStdio.reserve(poFdStdio.size() + poStreamStdio.size()); poStdio.reserve(poFdStdio.size() + poStreamStdio.size());
poStdio.insert(poStdio.begin(), poFdStdio.cbegin(), poFdStdio.cend()); poStdio.insert(poStdio.begin(), poFdStdio.cbegin(), poFdStdio.cend());
@ -280,9 +269,6 @@ public:
auto actual = FileHandle::Type{fd}; auto actual = FileHandle::Type{fd};
if(actual == FileHandle::Type{StdIN}) {
poFdStdio[0].flags = fgs;
} else {
auto it = std::find_if(poFdStdio.begin(), poFdStdio.end(), [actual](auto &&container){ auto it = std::find_if(poFdStdio.begin(), poFdStdio.end(), [actual](auto &&container){
return container.data.fd == actual; return container.data.fd == actual;
}); });
@ -296,7 +282,6 @@ public:
it->flags = fgs; it->flags = fgs;
it->data.fd = actual; it->data.fd = actual;
} }
}
return *this; return *this;
} }