some old code that was deleted used to look to see if the exit code was 0 and if so, returns a non-zero? The code that is deleted in a few places used to look like this - I'm wondering why we don't need to do that weird "if 0 then change retVal to 1" code?

-            if (rValue == RHQControl.EXIT_CODE_OK) {
-                // if somehow we were told it was OK, change it to unknown since it can't be OK
-                rValue = RHQControl.EXIT_CODE_STATUS_UNKNOWN;