memcached plugin: Clarify documentation and a comment.
authorFlorian Forster <octo@collectd.org>
Wed, 5 Oct 2016 06:51:29 +0000 (08:51 +0200)
committerFlorian Forster <octo@collectd.org>
Wed, 5 Oct 2016 06:51:29 +0000 (08:51 +0200)
src/collectd.conf.pod
src/memcached.c

index f867e61..a84bd40 100644 (file)
@@ -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>
 
index 46c8df4..765a0fd 100644 (file)
@@ -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)
   {