Merge pull request #16 from catharsis/master

commands: Inspect correct option for passive host check acceptance
commands: Let positive values of sticky mean yes

The acknowledge commands takes a "sticky" parameter. Before this patch,
0, 1, and >3 meant "not sticky", while 2 meant "sticky". I know why this
happened - internally, nagios uses 0 for "not acknowledged" and 1 for
"acknowledged", so internally, 2 is the first number available to use as
"stickily acknowledged". But. Really?

This changes the host and service acknowledge commands so that <=0 means
no and >0 means yes, the way all other booleans work. This does break
userspace API - though I suspect anyone using positive values other than
2 still intend for them to mean true, and are scratching their heads as
to why they don't work, as in op5#7794 or op5#5438.

Change-Id: Ib440df797b63c18d440cc1802bcce03b32341968
Signed-off-by: Robin Sonefors <robin.sonefors@op5.com>
naemon/naemon.c: removed uneeded else statement

to make code cleaner, removed an uneeded `else'
statement on naemon's main flow

Signed-off-by: Ricardo Maraschini <ricardo.maraschini@gmail.com>
naemon/sretention.c: removing uneeded comparsion

xrddefault_read_state_information() always returns OK or ERROR

Signed-off-by: Ricardo Maraschini <ricardo.maraschini@gmail.com>
Merge pull request #18 from ricardomaraschini/ricardo-patches

naemon/naemon.c: removed uneeded else statement
naemon/workers.c: removed uneeded variable attribution

`i' is set to zero a few lines below

Signed-off-by: Ricardo Maraschini <ricardo.maraschini@gmail.com>
fixed pkill on debian wheezy (fixes #17) (snarf)

Signed-off-by: Sven Nierlein <sven@consol.de>
Merge pull request #19 from ricardomaraschini/patches

naemon/workers.c: removed uneeded variable attribution