[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Chicken-hackers] [PATCH] Fix #819 by marking file-mkstemp as unimpl
From: |
Michele La Monaca |
Subject: |
Re: [Chicken-hackers] [PATCH] Fix #819 by marking file-mkstemp as unimplemented on Windows |
Date: |
Sun, 3 Aug 2014 01:45:22 +0200 |
On Sat, Aug 2, 2014 at 3:10 PM, Peter Bex <address@hidden> wrote:
> Hi all,
>
> This is an old bug and I think it's a good idea to remove file-mkstemp
> from the Windows version of the posix unit. It's been broken from the
> beginning, and nobody noticed. Maybe nobody is using it (I hope!), but
> it could also be that people are using it and expecting it to return
> sensible values.
>
> In any case, I opted for removing it because this is just extremely
> tricky to get right. We're better off removing it than keeping the
> broken version, so if you prefer to keep the procedure and want to
> do it properly, please write a proper patch.
I think this function is too important to be dismissed so easily.
Attached an implementation which gets rid of the limited mktemp
function. Actually a quick untested hack (I don't have access to a
mingw setup right now) but I hope a step in the right direction.
Regards,
Michele
0001-fix-file-mkstemp-on-windows-819.patch
Description: Binary data
Re: [Chicken-hackers] [PATCH] Fix #819 by marking file-mkstemp as unimplemented on Windows,
Michele La Monaca <=
Re: [Chicken-hackers] [PATCH] Fix #819 by marking file-mkstemp as unimplemented on Windows, Peter Bex, 2014/08/03
Re: [Chicken-hackers] [PATCH] Fix #819 by marking file-mkstemp as unimplemented on Windows, Peter Bex, 2014/08/03
Re: [Chicken-hackers] [PATCH] Fix #819 by marking file-mkstemp as unimplemented on Windows, Evan Hanson, 2014/08/12
Re: [Chicken-hackers] [PATCH] Fix #819 by marking file-mkstemp as unimplemented on Windows, Peter Bex, 2014/08/16