update
authorChristian Grothoff <christian@grothoff.org>
Tue, 27 Sep 2011 12:50:55 +0000 (12:50 +0000)
committerChristian Grothoff <christian@grothoff.org>
Tue, 27 Sep 2011 12:50:55 +0000 (12:50 +0000)
src/dht/dht_api_new.c
src/dht/gnunet-service-dht_neighbours.c

index 3fdd79a85572ab2805aad87ec4ca222b389ea821..9f7097d3459920d3ee19e5fa016283466662d577 100644 (file)
@@ -137,7 +137,7 @@ struct GNUNET_DHT_RouteHandle
   GNUNET_HashCode key;
 
   /**
-   * Unique identifier for this request (for key collisions). FIXME: redundant!?
+   * Unique identifier for this request (for key collisions).
    */
   uint64_t uid;
 
index 705168b17b02aee240cf74590da1edbc7ff4c289..cebf0e91f4778765d9e2c39d234206479e7d122e 100644 (file)
@@ -1763,6 +1763,8 @@ handle_dht_p2p_result (void *cls, const struct GNUNET_PeerIdentity *peer,
   data = (const void*) &get_path[get_path_length];
   data_size = msize - (sizeof (struct PeerResultMessage) + 
                       (get_path_length + put_path_length) * sizeof (struct GNUNET_PeerIdentity));
+
+  /* if we got a HELLO, consider it for our own routing table */
   if (type == GNUNET_BLOCK_TYPE_DHT_HELLO)
   {
     const struct GNUNET_MessageHeader *h;
@@ -1850,7 +1852,11 @@ process_hello (void *cls,
               const struct GNUNET_HELLO_Message *
               hello, const char *err_msg)
 {
-  // FIXME: track HELLOs, possibly ask core to establish connections
+  // FIXME: consider moving HELLO processing to another file!
+  // FIXME: first, filter HELLOs without addresses (!)
+  // FIXME: track HELLOs (for responding to FIND PEER requests)
+  // FIXME: add code to possibly ask core to establish connections
+  //        (using our own peerinfo is better than using FIND PEER!)
 }