[Nagios-users] Revamp of lib/worker.c

2013-03-16 Thread Andreas Ericsson
I know it's fairly late in the dev cycle, but the current status of 
lib/worker.c simply isn't maintainable enough. Among the several 
problems with it are:
  * spawn_worker() makes assumptions on how the worker manager wants to 
store the jobs it has sent to the worker.
  * More types than necessary are made public, which makes patching and 
minor feature addition close to impossible.
  * There is no common prefix used for functions and types, making it 
hard to remember which functions are public and which are private.
  * It's hard to test since it ties in pretty hard with the Nagios code.

I'll be doing quite an extensive revamp over the next few days in order 
to address these issues. This means base/workers.c will also sustain 
heavy surgery. If you're writing or have written modules or 
special-purpose workers that rely on any of the code about to be 
chainsaw-operated on, please let me know asap, and I'll do what I can to 
make sure your stuff keeps working. Be quick though, cause this needs to 
be done sharpish.

Thanks, and sorry for crossposting.

-- 
Andreas Ericsson   andreas.erics...@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.

--
Everyone hates slow websites. So do we.
Make your web apps faster with AppDynamics
Download AppDynamics Lite for free today:
http://p.sf.net/sfu/appdyn_d2d_mar
___
Nagios-users mailing list
Nagios-users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nagios-users
::: Please include Nagios version, plugin version (-v) and OS when reporting 
any issue. 
::: Messages without supporting info will risk being sent to /dev/null


[Nagios-users] Revamp of lib/worker.c

2013-03-16 Thread Andreas Ericsson
I know it's fairly late in the dev cycle, but the current status of 
lib/worker.c simply isn't maintainable enough. Among the several 
problems with it are:
  * spawn_worker() makes assumptions on how the worker manager wants to 
store the jobs it has sent to the worker.
  * More types than necessary are made public, which makes patching and 
minor feature addition close to impossible.
  * There is no common prefix used for functions and types, making it 
hard to remember which functions are public and which are private.
  * It's hard to test since it ties in pretty hard with the Nagios code.

I'll be doing quite an extensive revamp over the next few days in order 
to address these issues. This means base/workers.c will also sustain 
heavy surgery. If you're writing or have written modules or 
special-purpose workers that rely on any of the code about to be 
chainsaw-operated on, please let me know asap, and I'll do what I can to 
make sure your stuff keeps working. Be quick though, cause this needs to 
be done sharpish.

Thanks, and sorry for crossposting.

-- 
Andreas Ericsson   andreas.erics...@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.

--
Everyone hates slow websites. So do we.
Make your web apps faster with AppDynamics
Download AppDynamics Lite for free today:
http://p.sf.net/sfu/appdyn_d2d_mar
___
Nagios-users mailing list
Nagios-users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nagios-users
::: Please include Nagios version, plugin version (-v) and OS when reporting 
any issue. 
::: Messages without supporting info will risk being sent to /dev/null