remove getSockAddr() per Jeffs PR CR, add additional notes to reduce confusion

include/llarp/net.hpp

@@ -41,6 +41,8 @@ operator==(const in6_addr& a, const in6_addr& b);
 
 struct privatesInUse
 {
+  // true if used by real NICs on start
+  // false if not used, and means we could potentially use it if needed
   bool ten;       // 16m ips
   bool oneSeven;  // 1m  ips
   bool oneNine;   // 65k ips
@@ -446,12 +448,6 @@ namespace llarp
       return addr4()->s_addr;
     }
 
-    sockaddr*
-    getSockAddr()
-    {
-      return (struct sockaddr*)&_addr4;
-    }
-
     bool
     sameAddr(const Addr& other) const
     {

llarp/dns_dotlokilookup.cpp

@@ -110,7 +110,7 @@ llarp_dotlokilookup_checkQuery(void *u, uint64_t orig, uint64_t left)
     response->dontSendResponse         = false;
     // llarp::Addr test(*free_private->hostResult.getSockAddr());
     // llarp::LogInfo("IP Test: ", test);
-    response->returnThis = free_private->hostResult.getSockAddr();
+    response->returnThis = free_private->hostResult;
     llarp::LogInfo("Saving ", qr->request->question.name);
     loki_tld_lookup_cache[qr->request->question.name] = response;
     // we can't delete response now...

llarp/dnsd.cpp

@@ -269,7 +269,7 @@ handle_dnsc_result(dnsc_answer_request *client_request)
   else
   {
     writesend_dnss_response(
-        client_request->found ? client_request->result.getSockAddr() : nullptr,
+        client_request->found ? client_request->result : nullptr,
         server_request->from, server_request);
   }
   llarp_host_resolved(client_request);

llarp/net.cpp

@@ -807,6 +807,7 @@ struct privatesInUse
 llarp_getPrivateIfs()
 {
   struct privatesInUse result;
+  // mark all available for use
   result.ten      = false;
   result.oneSeven = false;
   result.oneNine  = false;