[MLton] Re: [MLton-commit] r6699

[Wesley W. Terpstra]

On Thu, Jun 18, 2009 at 4:34 AM, Matthew Fluet <fluet at tti-c.org> wrote:

>  MLton.Process.create (on Cygwin) has used cygwin_conv_to_full_win32_path
> since your 20041202 patch


Oh. :) From your examples I thought it was using windows paths, not cygwin.

But, I'm not convinced that it is worth it.


That's a reasonable position. The whole useWindowsProcess was implemented
AFAIK because we were using VirtualAlloc instead of mmap. At this pointis
there still any advantage to using VitrualAlloc? Use-mmap seems to work
quite reliably. It might be reasonable to just use fork()/exec() for
Process.create and eliminate the native memory management for cygwin.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mlton.org/pipermail/mlton/attachments/20090618/37b64535/attachment.html