diff options
author | ridiculousfish <corydoras@ridiculousfish.com> | 2016-02-28 01:38:28 -0800 |
---|---|---|
committer | ridiculousfish <corydoras@ridiculousfish.com> | 2016-02-28 01:38:28 -0800 |
commit | 9151ec709258273a555180696bdbae8bb0794658 (patch) | |
tree | be7eb6824e18216c71064227d277a0a708ba6ae5 /src/proc.cpp | |
parent | 3633c51ad8aee79add7727457c998c42a1638f72 (diff) |
Eliminate narrow_string_rep_t
This was used to cache a narrow string representation
of commands, so that if certain system calls returned errors
after fork, we could output error messages without allocating
memory. But in practice these errors are very uncommon, as are
commands that have wide characters. It is simpler to do a best-effort
output of the wide string, instead of caching a narrow string
unconditionally.
Diffstat (limited to 'src/proc.cpp')
-rw-r--r-- | src/proc.cpp | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/src/proc.cpp b/src/proc.cpp index be106d43..806da0ec 100644 --- a/src/proc.cpp +++ b/src/proc.cpp @@ -453,11 +453,7 @@ static void handle_child_status(pid_t pid, int status) } process_t::process_t() : - argv_array(), - argv0_narrow(), - type(), internal_block_node(NODE_OFFSET_INVALID), - actual_cmd(), pid(0), pipe_write_fd(0), pipe_read_fd(0), @@ -480,8 +476,6 @@ process_t::~process_t() } job_t::job_t(job_id_t jobid, const io_chain_t &bio) : - command_str(), - command_narrow(), block_io(bio), first_process(NULL), pgid(0), |