services: Remove an unused variable.
Francois Gouget
fgouget at codeweavers.com
Sat Aug 27 03:40:53 CDT 2011
On Fri, 26 Aug 2011, Frédéric Delanoy wrote:
[...]
> > - DWORD err;
> > const WCHAR *argv[2];
> > service = services_list[i];
> > argv[0] = service->name;
> > argv[1] = NULL;
> > - err = service_start(service, 1, argv);
> > + service_start(service, 1, argv);
> > /* FIXME: do something if the service failed to start */
> > release_service(service);
> > }
>
> Wouldn't it be better to do something with err rather than muting (as
> the comment suggests)?
Maybe but what? The FIXME does not even say. In the meantime a FIXME
comment is not a good enough reason to keep a compilation warning imho.
And conversely fixing a compilation warning is not 'muting' a FIXME
comment.
--
Francois Gouget <fgouget at codeweavers.com>
More information about the wine-devel
mailing list