Skip to content

Commit f595dcc

Browse files
sawenzelclaude
andcommitted
Fix the merger exit-status check in o2-sim
This fixes a problem in how the o2-sim driver judges the hit merger's exit code. - The condition `!= 0 || != 128` is always true, so every normal merger exit was reported as an error. - It now reads `!= 0 && != 128`. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
1 parent cd59f17 commit f595dcc

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

‎run/o2sim_parallel.cxx‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -763,7 +763,7 @@ int main(int argc, char* argv[])
763763
// Handle mergerpid status separately
764764
if (cpid == mergerpid) {
765765
if (WIFEXITED(status)) {
766-
if (WEXITSTATUS(status) != 0 || WEXITSTATUS(status) != 128) {
766+
if (WEXITSTATUS(status) != 0 && WEXITSTATUS(status) != 128) {
767767
LOG(error) << "Merger process exited with abnormal exit status " << WEXITSTATUS(status);
768768
errored = true;
769769
}

0 commit comments

Comments
 (0)