dlls/msvcrt: process.c (msvcrt-popen-1)
Jaco Greeff
jaco at puxedo.org
Fri Nov 1 12:11:09 CST 2002
On 01 Nov 2002 08:40:20 -0800, Alexandre Julliard <julliard at winehq.com> wrote :
> Jaco Greeff <jaco at puxedo.org> writes:
>
> > 1. Calling the popen function returns a FILE* which needs to be
> > converted to a MSVCRT_FILE*. The apprach taken here is potentially not
> > the best route but the only one I could think of. Is it suffiecient?
>
> No, it's wrong. You should be using the Windows API (CreateProcess
> etc.) not the Unix popen(). Then you can build a proper MSVCRT_FILE.
I was actually taking a re-look at that this afternoon. I agree, the initial
reason for th implementation was obviously that _popen is supposed to mimic
popen, but the file conversion leaves me feeling really uncomfortable.
I'll fix it up and submit it shortly.
Greetings,
Jaco
More information about the wine-devel
mailing list