squash? Improve readability - no change in behavior.

server/delay_server/src/irodsDelayServer.cpp

@@ -383,17 +383,17 @@ namespace
                 // Continue with given delay regardless of success
                 return update_rule_exec_info(false);
             case 1:
-                // Remove if successful, otherwise update next exec time
-                return !_exec_status ? delete_rule_exec_info() : update_rule_exec_info(false);
+                // Remove if successful (status == 0), otherwise update next exec time
+                return (0 == _exec_status) ? delete_rule_exec_info() : update_rule_exec_info(false);
             case 2:
                 // Remove regardless of success
                 return delete_rule_exec_info();
             case 3:
                 // Update with new exec time and frequency regardless of success
                 return update_rule_exec_info(true);
             case 4:
-                // Delete if successful, otherwise update with new exec time and frequency
-                return !_exec_status ? delete_rule_exec_info() : update_rule_exec_info(true);
+                // Remove if successful (status == 0), otherwise update with new exec time and frequency
+                return (0 == _exec_status) ? delete_rule_exec_info() : update_rule_exec_info(true);
             default:
                 log_ds::error("{}:{} - getNextRepeatTime returned unknown value {} for rule ID {}",
                                             __func__, __LINE__, repeat_status, _inp.ruleExecId);