commit 189eb5020bcffa7a7e601edf2fecb98008acfa9e
author Bernie Innocenti <codewiz@google.com> Mon Oct 01 23:10:18 2018 +0900
committer Bernie Innocenti <codewiz@google.com> Thu Oct 18 09:30:06 2018 +0000
tree 5c15b5f5e8b201982cde0cf2ec9a5776cebc2bcf
parent e03aed11e6ce5779873594c38694f72ca68c97e7

Move resolver public headers to separate include path

Additionally rename functions and structs to eliminate those ugly
underscores from the public API (with the notable exception of
__res_params, which will come in its own separate change to keep this
one from growing too large).

Test: build, flash, then 'atest netd_integration_test'
Change-Id: I7e9b0ee6cdbec889e9c35b5a17b5daa65533686e

resolv/res_send.cpp

diff --git a/resolv/res_send.cpp b/resolv/res_send.cpp
index 95605f4..4f73b59 100644
--- a/resolv/res_send.cpp
+++ b/resolv/res_send.cpp
@@ -100,12 +100,12 @@
 
 #include <android-base/logging.h>
 
+#include "netd_resolv/resolv.h"
+#include "netd_resolv/stats.h"
 #include "private/android_filesystem_config.h"
-#include "res_private.h"
+#include "res_state_ext.h"
 #include "resolv_cache.h"
-#include "resolv_netid.h"
 #include "resolv_private.h"
-#include "resolv_stats.h"
 
 #define EXT(res) ((res)->_u._ext)
 
@@ -501,9 +501,9 @@
      * Send request, RETRY times, or until successful.
      */
     for (int attempt = 0; attempt < statp->retry; ++attempt) {
-        struct __res_stats stats[MAXNS];
+        struct res_stats stats[MAXNS];
         struct __res_params params;
-        int revision_id = _resolv_cache_get_resolver_stats(statp->netid, &params, stats);
+        int revision_id = resolv_cache_get_resolver_stats(statp->netid, &params, stats);
         bool usable_servers[MAXNS];
         android_net_res_stats_get_usable_servers(&params, stats, statp->nscount, usable_servers);
 
@@ -572,7 +572,7 @@
                  * SERVFAIL or times out) do not unduly affect the stats.
                  */
                 if (attempt == 0) {
-                    struct __res_sample sample;
+                    res_sample sample;
                     _res_stats_set_sample(&sample, now, rcode, delay);
                     _resolv_cache_add_resolver_stats_sample(statp->netid, revision_id, ns, &sample,
                                                             params.max_samples);
@@ -592,7 +592,7 @@
 
                 /* Only record stats the first time we try a query. See above. */
                 if (attempt == 0) {
-                    struct __res_sample sample;
+                    res_sample sample;
                     _res_stats_set_sample(&sample, now, rcode, delay);
                     _resolv_cache_add_resolver_stats_sample(statp->netid, revision_id, ns, &sample,
                                                             params.max_samples);