[PATCH 2/2] lib/worker: Remove dead code

Andreas Ericsson ae at op5.se
Fri Nov 2 12:31:31 CET 2012


On 11/01/2012 05:38 PM, rsonefors at op5.com wrote:
> From: Robin Sonefors <robin.sonefors at op5.com>
> 
> This removes a method that was part of the public lib/worker API that
> hasn't been used since nagios started using its own query handler for
> spawning workers. It was highly specific to the old worker spawning
> code, meaning it's of little general use.
> 
> It also removes two members of the worker ABI that always pointed to
> NULL, which uses up (a little) RAM, and (a little more, due to confusion
> when examining core dumps) developer time.
> 
> Signed-off-by: Robin Sonefors <robin.sonefors at op5.com>

Declined. The lib/wproc binary still uses this function, so it's a no-go
for the time being. Moving it to lib/wproc.c would make sense, but I'm
not going to bother with that right now.

-- 
Andreas Ericsson                   andreas.ericsson at op5.se
OP5 AB                             www.op5.se
Tel: +46 8-230225                  Fax: +46 8-230231

Considering the successes of the wars on alcohol, poverty, drugs and
terror, I think we should give some serious thought to declaring war
on peace.

------------------------------------------------------------------------------
LogMeIn Central: Instant, anywhere, Remote PC access and management.
Stay in control, update software, and manage PCs from one command center
Diagnose problems and improve visibility into emerging IT issues
Automate, monitor and manage. Do more in less time with Central
http://p.sf.net/sfu/logmein12331_d2d




More information about the Developers mailing list