From cf27e2041102bb17ed5a3c495a50f4207d29ca1c Mon Sep 17 00:00:00 2001
From: Florian Forster <octo@collectd.org>
Date: Wed, 5 Oct 2016 08:51:29 +0200
Subject: [PATCH] memcached plugin: Clarify documentation and a comment.

---
 src/collectd.conf.pod | 11 ++++++-----
 src/memcached.c       | 13 +++++++------
 2 files changed, 13 insertions(+), 11 deletions(-)

diff --git a/src/collectd.conf.pod b/src/collectd.conf.pod
index f867e61b..a84bd408 100644
--- a/src/collectd.conf.pod
+++ b/src/collectd.conf.pod
@@ -3361,17 +3361,18 @@ following options are allowed:
 
 Sets the B<host> field of dispatched values. Defaults to the global hostname
 setting.
-For backwards compatibility, values are also dispatched with the global hostname
-when B<Host> is set to "127.0.0.1" or "localhost" and B<Address> is not set.
+For backwards compatibility, values are also dispatched with the global
+hostname when B<Host> is set to B<127.0.0.1> or B<localhost> and B<Address> is
+not set.
 
 =item B<Address> I<Address>
 
-Hostname or IP to connect to. For backwards compatibility, it defaults to B<Host>
-value or to B<127.0.0.1> if it does not set.
+Hostname or IP to connect to. For backwards compatibility, defaults to the
+value of B<Host> or B<127.0.0.1> if B<Host> is unset.
 
 =item B<Port> I<Port>
 
-TCP-Port to connect to. Defaults to B<11211>.
+TCP port to connect to. Defaults to B<11211>.
 
 =item B<Socket> I<Path>
 
diff --git a/src/memcached.c b/src/memcached.c
index 46c8df40..765a0fd3 100644
--- a/src/memcached.c
+++ b/src/memcached.c
@@ -532,14 +532,15 @@ static int memcached_add_read_callback (memcached_t *st)
       (st->name != NULL) ? st->name : "__legacy__");
 
   /* If no <Address> used then:
-   * - connect to destination, specified by <Host>, if it presents.
-   * - Keep default hostname, if any:
-   *    - Legacy mode is used (no configuration options at all);
-   *    - "Host" option is not provided;
+   * - Connect to the destination specified by <Host>, if present.
+   *   If not, use the default address.
+   * - Use the default hostname (set st->host to NULL), if
+   *    - Legacy mode is used (no configuration options at all), or
+   *    - "Host" option is not provided, or
    *    - "Host" option is set to "localhost" or "127.0.0.1".
    *
-   * If <Address> used then host may be set to "localhost"
-   * or "127.0.0.1" explicitly.
+   * If <Address> used then host may be set to "localhost" or "127.0.0.1"
+   * explicitly.
    */
   if (st->connhost == NULL)
   {
-- 
2.30.2