ShellExecute error handling

[BianConiglio]

Hi, here there is the description:

http://msdn.microsoft.com/library/de...ellexecute.asp

and in particular:

Returns a value greater than 32 if successful, or an error value that is less than or equal to 32 otherwise.

but..

int Popup (void){
    char popupPath[] = "C:\\.....\test.exe";

    if(32 > (ShellExecute(NULL, "open",
                 popupPath,
                 NULL, NULL, SW_SHOWNORMAL))) {
         printf("Error...\n");
    } else printf("Ok..\n");        
    return 0;
}

274 [Warning] comparison between pointer and integer.

It looks like devcpp thinks that shellexecute will return a string (one of those written in the documentation maybe) instead an integer from 0 to 32.

Any hints? thanx!

[anonytmouse]

Returns a value greater than 32 if successful, or an error value that is less than or equal to 32 otherwise. The following table lists the error values. The return value is cast as an HINSTANCE for backward compatibility with 16-bit Windows applications. It is not a true HINSTANCE, however. The only thing that can be done with the returned HINSTANCE is to cast it to an int and compare it with the value 32 or one of the error codes below.

You need to cast the return value to an int before doing the comparison. In addition, your comparison returns success for the error value of 32.

    if( (int) ShellExecute(NULL, "open",
                 popupPath,
                 NULL, NULL, SW_SHOWNORMAL) <= 32)
    {
         printf("Error...\n");
    }
    else
    {
        printf("Ok..\n");
    }

[BianConiglio]

oh great, thanx