diff options
author | Christian Grothoff <christian@grothoff.org> | 2014-12-24 01:10:47 +0000 |
---|---|---|
committer | Christian Grothoff <christian@grothoff.org> | 2014-12-24 01:10:47 +0000 |
commit | f1f603c7d0b3f03dca46a4f313472288eb080eb1 (patch) | |
tree | 3a29966b02dfb83e0a8a8d5c42b3116380209fb0 /src/hostlist/test_gnunet_daemon_hostlist_learning.c | |
parent | 53cd5b8eda2fa8db86b0907a62a39598981d008a (diff) | |
download | gnunet-f1f603c7d0b3f03dca46a4f313472288eb080eb1.tar.gz gnunet-f1f603c7d0b3f03dca46a4f313472288eb080eb1.zip |
making GNUNET_SCHEDULER_cancel() perform in O(1) instead of O(n) to help or even fully address #3247
Diffstat (limited to 'src/hostlist/test_gnunet_daemon_hostlist_learning.c')
-rw-r--r-- | src/hostlist/test_gnunet_daemon_hostlist_learning.c | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/src/hostlist/test_gnunet_daemon_hostlist_learning.c b/src/hostlist/test_gnunet_daemon_hostlist_learning.c index 9f4d8e3d7..5f497f242 100644 --- a/src/hostlist/test_gnunet_daemon_hostlist_learning.c +++ b/src/hostlist/test_gnunet_daemon_hostlist_learning.c | |||
@@ -64,9 +64,9 @@ static char *current_adv_uri; | |||
64 | 64 | ||
65 | static const struct GNUNET_CONFIGURATION_Handle *cfg; | 65 | static const struct GNUNET_CONFIGURATION_Handle *cfg; |
66 | 66 | ||
67 | static GNUNET_SCHEDULER_TaskIdentifier timeout_task; | 67 | static struct GNUNET_SCHEDULER_Task * timeout_task; |
68 | 68 | ||
69 | static GNUNET_SCHEDULER_TaskIdentifier check_task; | 69 | static struct GNUNET_SCHEDULER_Task * check_task; |
70 | 70 | ||
71 | static struct PeerContext adv_peer; | 71 | static struct PeerContext adv_peer; |
72 | 72 | ||
@@ -83,10 +83,10 @@ static void | |||
83 | shutdown_testcase () | 83 | shutdown_testcase () |
84 | { | 84 | { |
85 | GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "Shutdown testcase....\n"); | 85 | GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "Shutdown testcase....\n"); |
86 | if (timeout_task != GNUNET_SCHEDULER_NO_TASK) | 86 | if (timeout_task != NULL) |
87 | { | 87 | { |
88 | GNUNET_SCHEDULER_cancel (timeout_task); | 88 | GNUNET_SCHEDULER_cancel (timeout_task); |
89 | timeout_task = GNUNET_SCHEDULER_NO_TASK; | 89 | timeout_task = NULL; |
90 | } | 90 | } |
91 | if (NULL != download_stats) | 91 | if (NULL != download_stats) |
92 | { | 92 | { |
@@ -113,10 +113,10 @@ shutdown_testcase () | |||
113 | GNUNET_STATISTICS_destroy (learn_peer.stats, GNUNET_NO); | 113 | GNUNET_STATISTICS_destroy (learn_peer.stats, GNUNET_NO); |
114 | learn_peer.stats = NULL; | 114 | learn_peer.stats = NULL; |
115 | } | 115 | } |
116 | if (check_task != GNUNET_SCHEDULER_NO_TASK) | 116 | if (check_task != NULL) |
117 | { | 117 | { |
118 | GNUNET_SCHEDULER_cancel (check_task); | 118 | GNUNET_SCHEDULER_cancel (check_task); |
119 | check_task = GNUNET_SCHEDULER_NO_TASK; | 119 | check_task = NULL; |
120 | } | 120 | } |
121 | if (NULL != current_adv_uri) | 121 | if (NULL != current_adv_uri) |
122 | { | 122 | { |
@@ -168,7 +168,7 @@ shutdown_testcase () | |||
168 | static void | 168 | static void |
169 | timeout_error (void *cls, const struct GNUNET_SCHEDULER_TaskContext *tc) | 169 | timeout_error (void *cls, const struct GNUNET_SCHEDULER_TaskContext *tc) |
170 | { | 170 | { |
171 | timeout_task = GNUNET_SCHEDULER_NO_TASK; | 171 | timeout_task = NULL; |
172 | GNUNET_log (GNUNET_ERROR_TYPE_ERROR, | 172 | GNUNET_log (GNUNET_ERROR_TYPE_ERROR, |
173 | "Timeout while executing testcase, test failed.\n"); | 173 | "Timeout while executing testcase, test failed.\n"); |
174 | timeout = GNUNET_YES; | 174 | timeout = GNUNET_YES; |
@@ -269,7 +269,7 @@ check_statistics (void *cls, const struct GNUNET_SCHEDULER_TaskContext *tc) | |||
269 | { | 269 | { |
270 | char *stat; | 270 | char *stat; |
271 | 271 | ||
272 | check_task = GNUNET_SCHEDULER_NO_TASK; | 272 | check_task = NULL; |
273 | if (0 != (tc->reason & GNUNET_SCHEDULER_REASON_SHUTDOWN)) | 273 | if (0 != (tc->reason & GNUNET_SCHEDULER_REASON_SHUTDOWN)) |
274 | return; | 274 | return; |
275 | GNUNET_asprintf (&stat, gettext_noop ("# advertised URI `%s' downloaded"), | 275 | GNUNET_asprintf (&stat, gettext_noop ("# advertised URI `%s' downloaded"), |