services: Issue a fixme when a service fails to start.

Octavian Voicu octavian.voicu at gmail.com
Thu Sep 1 08:20:50 CDT 2011


On Thu, Sep 1, 2011 at 4:02 PM, Francois Gouget <fgouget at codeweavers.com> wrote:
> @@ -295,7 +295,8 @@ static void scmdatabase_autostart_services(struct scmdatabase *db)
>         argv[0] = service->name;
>         argv[1] = NULL;
>         err = service_start(service, 1, argv);
> -        /* FIXME: do something if the service failed to start */
> +        if (err != ERROR_SUCCESS)
> +            WINE_FIXME("do something if the service failed to start %d\n", err);

Why not print something more informative? This looks nice and also
tells you which service did not start:
WINE_FIXME("Auto-start service %s failed to start: %d\n",
wine_dbgstr_w(service->name), err);

Octavian



More information about the wine-devel mailing list