diff options
author | Christian Grothoff <christian@grothoff.org> | 2015-02-02 22:25:48 +0000 |
---|---|---|
committer | Christian Grothoff <christian@grothoff.org> | 2015-02-02 22:25:48 +0000 |
commit | 69c3a52cfa679175da8f06bdcb8e2e4195465e44 (patch) | |
tree | 73109126cfcf1a807c5dc1e778f40b955fc959c5 /src/transport/test_transport_api_monitor_peers.c | |
parent | c42deb089faffd654e27bf661ce85d5c1bb38b7a (diff) | |
download | gnunet-69c3a52cfa679175da8f06bdcb8e2e4195465e44.tar.gz gnunet-69c3a52cfa679175da8f06bdcb8e2e4195465e44.zip |
avoid passing both PeerIdentity and Address (which contains PeerIdentity) if address is always present as then this is redundant
Diffstat (limited to 'src/transport/test_transport_api_monitor_peers.c')
-rw-r--r-- | src/transport/test_transport_api_monitor_peers.c | 14 |
1 files changed, 6 insertions, 8 deletions
diff --git a/src/transport/test_transport_api_monitor_peers.c b/src/transport/test_transport_api_monitor_peers.c index 81d2f0478..536507c47 100644 --- a/src/transport/test_transport_api_monitor_peers.c +++ b/src/transport/test_transport_api_monitor_peers.c | |||
@@ -386,20 +386,19 @@ start_cb (struct PeerContext *p, void *cls) | |||
386 | 386 | ||
387 | static void | 387 | static void |
388 | monitor1_cb (void *cls, | 388 | monitor1_cb (void *cls, |
389 | const struct GNUNET_PeerIdentity *peer, | ||
390 | const struct GNUNET_HELLO_Address *address, | 389 | const struct GNUNET_HELLO_Address *address, |
391 | enum GNUNET_TRANSPORT_PeerState state, | 390 | enum GNUNET_TRANSPORT_PeerState state, |
392 | struct GNUNET_TIME_Absolute state_timeout) | 391 | struct GNUNET_TIME_Absolute state_timeout) |
393 | { | 392 | { |
394 | if ((NULL == peer) || (NULL == p1)) | 393 | if ((NULL == address) || (NULL == p1)) |
395 | return; | 394 | return; |
396 | 395 | ||
397 | GNUNET_log (GNUNET_ERROR_TYPE_INFO, | 396 | GNUNET_log (GNUNET_ERROR_TYPE_INFO, |
398 | "Monitor 1: %s %s %s\n", | 397 | "Monitor 1: %s %s %s\n", |
399 | GNUNET_i2s (peer), | 398 | GNUNET_i2s (&address->peer), |
400 | GNUNET_TRANSPORT_ps2s (state), | 399 | GNUNET_TRANSPORT_ps2s (state), |
401 | GNUNET_STRINGS_absolute_time_to_string(state_timeout)); | 400 | GNUNET_STRINGS_absolute_time_to_string(state_timeout)); |
402 | if ((0 == memcmp (peer, &p2->id, sizeof (p2->id)) && | 401 | if ((0 == memcmp (&address->peer, &p2->id, sizeof (p2->id)) && |
403 | (GNUNET_YES == GNUNET_TRANSPORT_is_connected(state)) && | 402 | (GNUNET_YES == GNUNET_TRANSPORT_is_connected(state)) && |
404 | GNUNET_NO == p1_c) ) | 403 | GNUNET_NO == p1_c) ) |
405 | { | 404 | { |
@@ -412,20 +411,19 @@ monitor1_cb (void *cls, | |||
412 | 411 | ||
413 | static void | 412 | static void |
414 | monitor2_cb (void *cls, | 413 | monitor2_cb (void *cls, |
415 | const struct GNUNET_PeerIdentity *peer, | ||
416 | const struct GNUNET_HELLO_Address *address, | 414 | const struct GNUNET_HELLO_Address *address, |
417 | enum GNUNET_TRANSPORT_PeerState state, | 415 | enum GNUNET_TRANSPORT_PeerState state, |
418 | struct GNUNET_TIME_Absolute state_timeout) | 416 | struct GNUNET_TIME_Absolute state_timeout) |
419 | { | 417 | { |
420 | if ((NULL == peer) || (NULL == p2)) | 418 | if ((NULL == address) || (NULL == p2)) |
421 | return; | 419 | return; |
422 | 420 | ||
423 | GNUNET_log (GNUNET_ERROR_TYPE_INFO, | 421 | GNUNET_log (GNUNET_ERROR_TYPE_INFO, |
424 | "Monitor 2: %s %s %s\n", | 422 | "Monitor 2: %s %s %s\n", |
425 | GNUNET_i2s (peer), | 423 | GNUNET_i2s (&address->peer), |
426 | GNUNET_TRANSPORT_ps2s (state), | 424 | GNUNET_TRANSPORT_ps2s (state), |
427 | GNUNET_STRINGS_absolute_time_to_string(state_timeout)); | 425 | GNUNET_STRINGS_absolute_time_to_string(state_timeout)); |
428 | if ((0 == memcmp (peer, &p1->id, sizeof (p1->id)) && | 426 | if ((0 == memcmp (&address->peer, &p1->id, sizeof (p1->id)) && |
429 | (GNUNET_YES == GNUNET_TRANSPORT_is_connected(state)) && | 427 | (GNUNET_YES == GNUNET_TRANSPORT_is_connected(state)) && |
430 | GNUNET_NO == p2_c) ) | 428 | GNUNET_NO == p2_c) ) |
431 | { | 429 | { |