projects
/
deliverable
/
lttng-tools.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
Tests: Fix: `wait_on_file()` returns too early
[deliverable/lttng-tools.git]
/
tests
/
regression
/
tools
/
notification
/
notification.c
diff --git
a/tests/regression/tools/notification/notification.c
b/tests/regression/tools/notification/notification.c
index 19df9bc3049f47681b4c8f494d56e5da9bdcf0f0..f076bbd0e5f5373740061b13f47ef4efdfcd2ad9 100644
(file)
--- a/
tests/regression/tools/notification/notification.c
+++ b/
tests/regression/tools/notification/notification.c
@@
-59,16
+59,31
@@
void wait_on_file(const char *path, bool file_exist)
ret = stat(path, &buf);
if (ret == -1 && errno == ENOENT) {
if (file_exist) {
ret = stat(path, &buf);
if (ret == -1 && errno == ENOENT) {
if (file_exist) {
- (void) poll(NULL, 0, 10); /* 10 ms delay */
- continue; /* retry */
+ /*
+ * The file does not exist. wait a bit and
+ * continue looping until it does.
+ */
+ (void) poll(NULL, 0, 10);
+ continue;
}
}
- break; /* File does not exist */
+
+ /*
+ * File does not exist and the exit condition we want.
+ * Break from the loop and return.
+ */
+ break;
}
if (ret) {
perror("stat");
exit(EXIT_FAILURE);
}
}
if (ret) {
perror("stat");
exit(EXIT_FAILURE);
}
- break; /* found */
+ /*
+ * stat() returned 0, so the file exists. break now only if
+ * that's the exit condition we want.
+ */
+ if (file_exist) {
+ break;
+ }
}
}
}
}
This page took
0.027894 seconds
and
5
git commands to generate.