[PATCH] remove double definition of HAL_HELPER_TIMEOUT

Danny Kukawka danny.kukawka at web.de
Tue Oct 30 14:14:20 PDT 2007


Hi,

I couldn't see a compiler warning, but IMO the double definition of 
HAL_HELPER_TIMEOUT in hald/util.h and hald/hald_runner.h could cause problems 
if one of the both values get may changed in the future (currently both 
10000).

I would remove the define of HAL_HELPER_TIMEOUT from hald/util.h since the 
timeout is needed for hald_runner and where you use functions from this file 
you need to include already the header file hald_runner.h ...

Danny

 util.h |    1 -
 1 file changed, 1 deletion(-)
-------------- next part --------------
A non-text attachment was scrubbed...
Name: hal-fix_HAL_HELPER_TIMEOUT.diff
Type: text/x-diff
Size: 271 bytes
Desc: not available
Url : http://lists.freedesktop.org/archives/hal/attachments/20071030/737c2e1b/attachment.diff 


More information about the hal mailing list