process: Confusion about UseHandle handles being closed.

Felipe Lessa felipe.lessa at gmail.com
Tue Nov 25 13:54:42 UTC 2014


On 25-11-2014 08:39, 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. (Btw. why are the other fields
> named with underscores?) Otherwise, every new switch to createProcess
> multiplies the number of its variants. Users who create the
> CreateProcess structure using 'shell' and 'proc' won't need to adapt
> their code.

This would require a major version bump to the library.  It may not be
worthy for such a small change.

Cheers,

-- 
Felipe.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: OpenPGP digital signature
URL: <http://www.haskell.org/pipermail/libraries/attachments/20141125/f3beb5ca/attachment.sig>


More information about the Libraries mailing list