RE: [PATCH] cmd: Add basic handling of Ctrl-C

[Hugh McMaster]

On Thu, 26 Mar 2015 14:39:12 +0300, Nikolay Sivov wrote:
> On 26.03.2015 14:32, Hugh McMaster wrote:
> > +  /* Enable handling of Ctrl-C signals */
> > +  if (SetConsoleCtrlHandler((PHANDLER_ROUTINE)console_ctrl_handler, TRUE))
> > +      WINE_TRACE("cmd: Ctrl handler was installed.\n");
> > +  else
> > +      WINE_ERR("cmd: Unable to install Ctrl handler.\n");
> > +
>
> Judging by the fact that you needed a cast, that's not right.

The PHANDLER_ROUTINE is a pointer to the function handler. MSDN's own example [1] on installing a Ctrl handler uses a cast.

Otherwise, what would you suggest?

[1] https://msdn.microsoft.com/en-us/library/windows/desktop/ms685049%28v=vs.85%...