Download Reference Manual
The Developer's Library for D
About Wiki Forums Source Search Contact

Ticket #408 (closed wishlist: fixed)

Opened 1 year ago

Last modified 1 year ago

tango.sys.Process needs a variant env constructor

Reported by: afb Assigned to: jcomellas
Priority: normal Milestone: 0.98 RC 2
Component: Tango Version: trunk
Keywords: Cc:

Description

tango.sys.Process could do with a second constructor like:

public this(char[] command, char[char[]] environment)
{
    char[][] env;
    foreach (key, value; environment.get)
        env ~= (key ~ "=" ~ value);
    this(command, env);
}

So that you can pass Environment.get in, for instance ?

Change History

04/18/07 04:59:01 changed by larsivi

  • milestone set to 0.98 RC 2.

04/18/07 15:12:41 changed by jcomellas

  • status changed from new to assigned.

This makes much more sense than what we have now. My original plan was to use the current format to avoid memory allocations, but that is not happening. I will change all appearances of the 'env' array to an associative array. There is no sense in keeping the current interface.

04/18/07 18:03:09 changed by jcomellas

  • status changed from assigned to closed.
  • resolution set to fixed.

The requested changes have been committed to SVN in r2073. Please check out the latest version and let me know if there is any problem.

04/22/07 07:53:56 changed by afb

But no "deprecated" entry for the old / array param ?

Works OK, once the code was updated... (changed mine)