🚀 go-pugleaf

"Daniele Borini" <borini@infinito.it> wrote in message
news:40bb1e04@newsgroups.borland.com...

> this is my code:

Use this code instead:

    {
        HANDLE hStdInRead = NULL;
        HANDLE hStdInWrite = NULL;
        HANDLE hStdOutRead = NULL;
        HANDLE hStdOutWrite = NULL;

        if( CreatePipe(&hStdInRead, &hStdInWrite, NULL, 10000) )
        {
            if( CreatePipe(&hStdOutRead, &hStdOutWrite, NULL, 10000) )
            {
                STARTUPINFO sInfo = {0};
                PROCESSINFO pInfo = {0};

                sInfo.dwFlags = STARTF_USESHOWWINDOW | STARTF_USESTDHANDLES;
                sInfo.wShowWindow = SW_NORMAL;
                sInfo.hStdInput = hStdInRead;
                sInfo.hStdOutput = hStdOutWrite;
                sInfo.hStdError = hStdOutWrite;

                if( CreateProcess(NULL, "cmd.exe", NULL, NULL, TRUE,
NORMAL_PRIORITY_CLASS, NULL, NULL, &sInfo, &pInfo) )
                {
                    WriteFile(hStdInWrite, dir *.*\n, 8, &i, NULL);

                    DWORD i = 0;
                    char str_out[10000] = {0};
                    ReadFile(hStdOutRead, str_out, sizeof(str_out), &i,
NULL);

                    Display->Lines->Add(str_out);
                }
            }
        }
    }

Even better, for a directory listing you really should be using
FindFirstFile(), FindNextFile(), and FileClose() instead.  Then you don't
need CreateProcess() at all.


Gambit