Skip to content

Commit

Permalink
i#2845: fix racy nudge_test test output (#2846)
Browse files Browse the repository at this point in the history
Fixes the nudge_test output to not have two "done" message that result in a
race in runall.cmake.

Issue: #1309
Fixes #2845
  • Loading branch information
derekbruening authored Feb 22, 2018
1 parent b726e89 commit 4b2eec1
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 1 deletion.
3 changes: 3 additions & 0 deletions suite/tests/client-interface/nudge_test.template
Original file line number Diff line number Diff line change
@@ -1,4 +1,7 @@
thank you for testing the client interface
nudge delivered 10
nudge delivered 11
#ifdef WINDOWS
MessageBox closed
#endif
done
2 changes: 1 addition & 1 deletion suite/tests/win32/infloop.c
Original file line number Diff line number Diff line change
Expand Up @@ -54,6 +54,6 @@ main(int argc, const char *argv[])
"Infloop pid=%d", GetProcessId(GetCurrentProcess()));
SetTimer(NULL, NULL, 180*1000/*3 mins*/, TimerProc);
MessageBoxA(NULL, "DynamoRIO test: will be auto-closed", title, MB_OK);
print("done\n");
print("MessageBox closed\n");
return 0;
}

0 comments on commit 4b2eec1

Please sign in to comment.