CODEWITHSUNDEEP
cwindowsmingwcompiler-warnings
rom1v
rom1v

Reputation: 2979

Why calling SetConsoleCtrlHandler() triggers a warning?

#include <windows.h>

BOOL MyCtrlHandler(DWORD ctrlType) {
    return TRUE;
}

int main(void) {
    SetConsoleCtrlHandler(MyCtrlHandler, TRUE);
    return 0;
}

The function signature matches the doc:

The PHANDLER_ROUTINE type defines a pointer to this callback function. HandlerRoutine is a placeholder for the application-defined function name.

If I cross-compile it with MinGW in 64 bits, it works:

$ x86_64-w64-mingw32-gcc ctrl.c
$

But in 32 bits, I get a warning:

$ i686-w64-mingw32-gcc ctrl.c
ctrl.c: In function ‘main’:
ctrl.c:8:27: warning: passing argument 1 of ‘SetConsoleCtrlHandler’ from incompatible pointer type [-Wincompatible-pointer-types]
    8 |     SetConsoleCtrlHandler(MyCtrlHandler, TRUE);
      |                           ^~~~~~~~~~~~~
      |                           |
      |                           BOOL (*)(DWORD) {aka int (*)(long unsigned int)}
In file included from /usr/share/mingw-w64/include/windows.h:74,
                 from ctrl.c:1:
/usr/share/mingw-w64/include/wincon.h:249:68: note: expected ‘PHANDLER_ROUTINE’ but argument is of type ‘BOOL (*)(DWORD)’ {aka ‘int (*)(long unsigned int)’}
  249 |   WINBASEAPI WINBOOL WINAPI SetConsoleCtrlHandler(PHANDLER_ROUTINE HandlerRoutine,WINBOOL Add);
      |
$

I don't understand why, because MyCtrlHandler should have the same type as PHANDLER_ROUTINE.

Upvotes: 1

Views: 447

Answers (1)

jwdonahue
jwdonahue

Reputation: 6669

The problem here is the difference between __cdecl and __stdcall. You must have lucked out with the default calling convention on your one successful attempt to compile. You need:

BOOL WINAPI MyCtrlHandler(DWORD ctrlType) {
    return TRUE;
}

Upvotes: 4

Related Questions