Skip to content

Commit

Permalink
Fixed inconsistent usage of result_str and output
Browse files Browse the repository at this point in the history
Fixed inconsistent usage of result_str and output (pointed out by Philipp Wendler in his review).
  • Loading branch information
rmetta authored Nov 2, 2023
1 parent bacb3d3 commit bb769a8
Showing 1 changed file with 1 addition and 4 deletions.
5 changes: 1 addition & 4 deletions benchexec/tools/proton.py
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ def determine_result(self, run):
elif "FALSE(termination)" in result_str:
status = result.RESULT_FALSE_TERMINATION

elif "UNKNOWN" in output:
elif "UNKNOWN" in result_str:
status = result.RESULT_UNKNOWN

elif "INTERNAL-ERROR" in result_str:
Expand All @@ -59,7 +59,4 @@ def determine_result(self, run):
elif run.exit_code.value == 64 and "Usage error!" in output:
status = "INVALID ARGUMENTS"

else:
status = result.RESULT_ERROR

return status

0 comments on commit bb769a8

Please sign in to comment.