On Sat, 29 Nov 2014, Bertram Felgenhauer wrote:
Henning Thielemann wrote:
On Tue, 25 Nov 2014, Michael Snoyman wrote:
1. Add clear documentation to `createProcess` indicating that it will close the `Handle` automatically. 2. Add a new function to be exported from `System.Process` with the semantics of `createProcess_`. I'm open to bikeshedding on the name, but perhaps sticking with `createProcess_` makes the most sense.
I think the cleanest solution would be to add a new field (e.g. leaveOpen) to the CreateProcess record.
Wouldn't it be more natural to add a new constructor to the StdStream type?
Yes, this would allow to choose a closing procedure per stream. (Nonetheless, a smart setter function 'leaveOpen' could turn all StdStream fields to ShareHandle.)