Provide /stats F to report even defaulted feature values.
[ircu2.10.12-pk.git] / ircd / ircd_features.c
index 9fe5195719023bf7a7454f6d0104700dd94abdf9..baaa2bd0e146c1763d735681c8bf57bffed8060c 100644 (file)
  * You should have received a copy of the GNU General Public License
  * along with this program; if not, write to the Free Software
  * Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
- *
- * $Id$
+ */
+/** @file
+ * @brief Implementation of configurable feature support.
+ * @version $Id$
  */
 #include "config.h"
 
 #include "s_bsd.h"
 #include "s_debug.h"
 #include "s_misc.h"
+#include "s_stats.h"
 #include "send.h"
 #include "struct.h"
-#include "support.h"
 #include "sys.h"    /* FALSE bleah */
 #include "whowas.h"    /* whowas_realloc */
 
-#include <assert.h>
+/* #include <assert.h> -- Now using assert in ircd_log.h */
 #include <stdlib.h>
 #include <string.h>
 
-/* List of log output types that can be set */
+struct Client his;
+
+/** List of log output types that can be set */
 static struct LogTypes {
-  char *type;
-  int (*set)(const char *, const char *);
-  char *(*get)(const char *);
+  char *type; /**< Settable name. */
+  int (*set)(const char *, const char *); /**< Function to set the value. */
+  char *(*get)(const char *); /**< Function to get the value. */
 } logTypes[] = {
   { "FILE", log_set_file, log_get_file },
   { "FACILITY", log_set_facility, log_get_facility },
@@ -62,7 +66,11 @@ static struct LogTypes {
   { 0, 0, 0 }
 };
 
-/* Look up a struct LogType given the type string */
+/** Look up a struct LogType given the type string.
+ * @param[in] from &Client requesting type, or NULL.
+ * @param[in] type Name of log type to find.
+ * @return Pointer to the found LogType, or NULL if none was found.
+ */
 static struct LogTypes *
 feature_log_desc(struct Client* from, const char *type)
 {
@@ -83,7 +91,12 @@ feature_log_desc(struct Client* from, const char *type)
   return 0; /* not found */
 }
 
-/* Set the value of a log output type for a log subsystem */
+/** Set the value of a log output type for a log subsystem.
+ * @param[in] from &Client trying to set the log type, or NULL.
+ * @param[in] fields Array of parameters to set.
+ * @param[in] count Number of parameters in \a fields.
+ * @return -1 to clear the mark, 0 to leave the mask alone, 1 to set the mask.
+ */
 static int
 feature_log_set(struct Client* from, const char* const* fields, int count)
 {
@@ -123,7 +136,12 @@ feature_log_set(struct Client* from, const char* const* fields, int count)
   return 0;
 }
 
-/* reset a log type for a subsystem to its default value */
+/** Reset a log type for a subsystem to its default value.
+ * @param[in] from &Client trying to reset the subsystem.
+ * @param[in] fields Array of parameters to reset.
+ * @param[in] count Number of fields in \a fields.
+ * @return -1 to unmark the entry, or zero to leave it alone.
+ */
 static int
 feature_log_reset(struct Client* from, const char* const* fields, int count)
 {
@@ -145,7 +163,25 @@ feature_log_reset(struct Client* from, const char* const* fields, int count)
   return 0;
 }
 
-/* report the value of a log setting */
+/** Handle an update to FEAT_HIS_SERVERNAME. */
+static void
+feature_notify_servername(void)
+{
+  ircd_strncpy(cli_name(&his), feature_str(FEAT_HIS_SERVERNAME), HOSTLEN);
+}
+
+/** Handle an update to FEAT_HIS_SERVERINFO. */
+static void
+feature_notify_serverinfo(void)
+{
+  ircd_strncpy(cli_info(&his), feature_str(FEAT_HIS_SERVERINFO), REALLEN);
+}
+
+/** Report the value of a log setting.
+ * @param[in] from &Client asking for details.
+ * @param[in] fields Array of parameters to get.
+ * @param[in] count Number of fields in \a fields.
+ */
 static void
 feature_log_get(struct Client* from, const char* const* fields, int count)
 {
@@ -172,64 +208,94 @@ feature_log_get(struct Client* from, const char* const* fields, int count)
   }
 }
 
-/* sets a feature to the given value */
-typedef int  (*feat_set_call)(struct Client*, const char* const*, int);
-/* gets the value of a feature */
-typedef void (*feat_get_call)(struct Client*, const char* const*, int);
-/* callback to notify of a feature's change */
+/** Update whether #me is a hub or not.
+ */
+static void
+feature_notify_hub(void)
+{
+  if (feature_bool(FEAT_HUB))
+    SetHub(&me);
+  else
+    ClearHub(&me);
+}
+
+/** Sets a feature to the given value.
+ * @param[in] from Client trying to set parameters.
+ * @param[in] fields Array of parameters to set.
+ * @param[in] count Number of fields in \a count.
+ * @return <0 to clear the feature mark, 0 to leave it, >0 to set the feature mark.
+ */
+typedef int  (*feat_set_call)(struct Client* from, const char* const* fields, int count);
+/** Gets the value of a feature.
+ * @param[in] from Client trying to get parameters.
+ * @param[in] fields Array of parameters to set.
+ * @param[in] count Number of fields in \a count.
+ */
+typedef void (*feat_get_call)(struct Client* from, const char* const* fields, int count);
+/** Callback to notify of a feature's change. */
 typedef void (*feat_notify_call)(void);
-/* unmarks all sub-feature values prior to reading .conf */
+/** Unmarks all sub-feature values prior to reading .conf. */
 typedef void (*feat_unmark_call)(void);
-/* resets to defaults all currently unmarked values */
-typedef int  (*feat_mark_call)(int);
-/* reports features as a /stats f list */
-typedef void (*feat_report_call)(struct Client*, int);
+/** Resets to defaults all currently unmarked values.
+ * @param[in] marked Non-zero if the feature is marked.
+ */
+typedef int  (*feat_mark_call)(int marked);
+/* Reports features as a /stats f list.
+ * @param[in] sptr Client asking for feature list.
+ * @param[in] marked Non-zero if the feature is marked.
+ */
+typedef void (*feat_report_call)(struct Client* sptr, int marked);
 
-#define FEAT_NONE   0x0000     /* no value */
-#define FEAT_INT    0x0001     /* set if entry contains an integer value */
-#define FEAT_BOOL   0x0002     /* set if entry contains a boolean value */
-#define FEAT_STR    0x0003     /* set if entry contains a string value */
-#define FEAT_MASK   0x000f     /* possible value types */
+#define FEAT_NONE   0x0000     /**< no value */
+#define FEAT_INT    0x0001     /**< set if entry contains an integer value */
+#define FEAT_BOOL   0x0002     /**< set if entry contains a boolean value */
+#define FEAT_STR    0x0003     /**< set if entry contains a string value */
+#define FEAT_MASK   0x000f     /**< possible value types */
 
-#define FEAT_MARK   0x0010     /* set if entry has been changed */
-#define FEAT_NULL   0x0020     /* NULL string is permitted */
-#define FEAT_CASE   0x0040     /* string is case-sensitive */
+#define FEAT_MARK   0x0010     /**< set if entry has been changed */
+#define FEAT_NULL   0x0020     /**< NULL string is permitted */
+#define FEAT_CASE   0x0040     /**< string is case-sensitive */
 
-#define FEAT_OPER   0x0100     /* set to display only to opers */
-#define FEAT_MYOPER 0x0200     /* set to display only to local opers */
-#define FEAT_NODISP 0x0400     /* feature must never be displayed */
+#define FEAT_OPER   0x0100     /**< set to display only to opers */
+#define FEAT_MYOPER 0x0200     /**< set to display only to local opers */
+#define FEAT_NODISP 0x0400     /**< feature must never be displayed */
 
-#define FEAT_READ   0x1000     /* feature is read-only (for now, perhaps?) */
+#define FEAT_READ   0x1000     /**< feature is read-only (for now, perhaps?) */
 
-static struct FeatureDesc {
-  enum Feature    feat;    /* feature identifier */
-  char*                   type;    /* string describing type */
-  unsigned int     flags;   /* flags for feature */
-  int             v_int;   /* integer value */
-  int             def_int; /* default value */
-  char*                   v_str;   /* string value */
-  char*                   def_str; /* default value */
-  feat_set_call           set;     /* set feature values */
-  feat_set_call           reset;   /* reset feature values to defaults */
-  feat_get_call           get;     /* get feature values */
-  feat_notify_call notify;  /* notify of value change */
-  feat_unmark_call unmark;  /* unmark all feature change values */
-  feat_mark_call   mark;    /* reset to defaults all unchanged features */
-  feat_report_call report;  /* report feature values */
-} features[] = {
+/** Declare a feature with custom behavior. */
 #define F_N(type, flags, set, reset, get, notify, unmark, mark, report)              \
   { FEAT_ ## type, #type, FEAT_NONE | (flags), 0, 0, 0, 0,                   \
     (set), (reset), (get), (notify), (unmark), (mark), (report) }
+/** Declare a feature that takes integer values. */
 #define F_I(type, flags, v_int, notify)                                              \
   { FEAT_ ## type, #type, FEAT_INT | (flags), 0, (v_int), 0, 0,                      \
     0, 0, 0, (notify), 0, 0, 0 }
+/** Declare a feature that takes boolean values. */
 #define F_B(type, flags, v_int, notify)                                              \
   { FEAT_ ## type, #type, FEAT_BOOL | (flags), 0, (v_int), 0, 0,             \
     0, 0, 0, (notify), 0, 0, 0 }
+/** Declare a feature that takes string values. */
 #define F_S(type, flags, v_str, notify)                                              \
   { FEAT_ ## type, #type, FEAT_STR | (flags), 0, 0, 0, (v_str),                      \
     0, 0, 0, (notify), 0, 0, 0 }
 
+/** Table of feature descriptions. */
+static struct FeatureDesc {
+  enum Feature    feat;    /**< feature identifier */
+  char*                   type;    /**< string describing type */
+  unsigned int     flags;   /**< flags for feature */
+  int             v_int;   /**< integer value */
+  int             def_int; /**< default value */
+  char*                   v_str;   /**< string value */
+  char*                   def_str; /**< default value */
+  feat_set_call           set;     /**< set feature values */
+  feat_set_call           reset;   /**< reset feature values to defaults */
+  feat_get_call           get;     /**< get feature values */
+  feat_notify_call notify;  /**< notify of value change */
+  feat_unmark_call unmark;  /**< unmark all feature change values */
+  feat_mark_call   mark;    /**< reset to defaults all unchanged features */
+  feat_report_call report;  /**< report feature values */
+} features[] = {
   /* Misc. features */
   F_N(LOG, FEAT_MYOPER, feature_log_set, feature_log_reset, feature_log_get,
       0, log_feature_unmark, log_feature_mark, log_feature_report),
@@ -244,7 +310,7 @@ static struct FeatureDesc {
   F_S(PROVIDER, FEAT_NULL, 0, 0),
   F_B(KILL_IPMISMATCH, FEAT_OPER, 0, 0),
   F_B(IDLE_FROM_MSG, 0, 1, 0),
-  F_B(HUB, 0, 0, 0),
+  F_B(HUB, 0, 0, feature_notify_hub),
   F_B(WALLOPS_OPER_ONLY, 0, 0, 0),
   F_B(NODNS, 0, 0, 0),
   F_N(RANDOM_SEED, FEAT_NODISP, random_seed_set, 0, 0, 0, 0, 0, 0),
@@ -255,8 +321,10 @@ static struct FeatureDesc {
   F_S(HIDDEN_IP, 0, "127.0.0.1", 0),
   F_B(CONNEXIT_NOTICES, 0, 0, 0),
   F_B(OPLEVELS, 0, 1, 0),
+  F_B(ZANNELS, 0, 1, 0),
   F_B(LOCAL_CHANNELS, 0, 1, 0),
   F_B(TOPIC_BURST, 0, 0, 0),
+  F_B(DISABLE_GLINES, 0, 0, 0),
 
   /* features that probably should not be touched */
   F_I(KILLCHASETIMELIMIT, 0, 30, 0),
@@ -268,17 +336,17 @@ static struct FeatureDesc {
   F_I(HANGONGOODLINK, 0, 300, 0),
   F_I(HANGONRETRYDELAY, 0, 10, 0),
   F_I(CONNECTTIMEOUT, 0, 90, 0),
-  F_I(TIMESEC, 0, 60, 0),
   F_I(MAXIMUM_LINKS, 0, 1, init_class), /* reinit class 0 as needed */
   F_I(PINGFREQUENCY, 0, 120, init_class),
   F_I(CONNECTFREQUENCY, 0, 600, init_class),
   F_I(DEFAULTMAXSENDQLENGTH, 0, 40000, init_class),
   F_I(GLINEMAXUSERCOUNT, 0, 20, 0),
-  F_I(SOCKSENDBUF, 0, 0, 0),
-  F_I(SOCKRECVBUF, 0, 0, 0),
+  F_I(SOCKSENDBUF, 0, SERVER_TCP_WINDOW, 0),
+  F_I(SOCKRECVBUF, 0, SERVER_TCP_WINDOW, 0),
   F_I(IPCHECK_CLONE_LIMIT, 0, 4, 0),
   F_I(IPCHECK_CLONE_PERIOD, 0, 40, 0),
   F_I(IPCHECK_CLONE_DELAY, 0, 600, 0),
+  F_I(CHANNELLEN, 0, 200, 0),
 
   /* Some misc. default paths */
   F_S(MPATH, FEAT_CASE | FEAT_MYOPER, "ircd.motd", motd_init),
@@ -286,36 +354,37 @@ static struct FeatureDesc {
   F_S(PPATH, FEAT_CASE | FEAT_MYOPER | FEAT_READ, "ircd.pid", 0),
 
   /* Networking features */
-  F_B(VIRTUAL_HOST, 0, 0, 0),
   F_I(TOS_SERVER, 0, 0x08, 0),
   F_I(TOS_CLIENT, 0, 0x08, 0),
   F_I(POLLS_PER_LOOP, 0, 200, 0),
   F_I(IRCD_RES_RETRIES, 0, 2, 0),
   F_I(IRCD_RES_TIMEOUT, 0, 4, 0),
   F_I(AUTH_TIMEOUT, 0, 9, 0),
+  F_B(ANNOUNCE_INVITES, 0, 0, 0),
 
   /* features that affect all operators */
-  F_B(CRYPT_OPER_PASSWORD, FEAT_MYOPER | FEAT_READ, 1, 0),
   F_B(CONFIG_OPERCMDS, 0, 0, 0),
-  F_B(LIST_CHAN, 0, 1, 0),
 
   /* HEAD_IN_SAND Features */
   F_B(HIS_SNOTICES, 0, 1, 0),
   F_B(HIS_SNOTICES_OPER_ONLY, 0, 1, 0),
-  F_B(HIS_DESYNCS, 0, 1, 0),
   F_B(HIS_DEBUG_OPER_ONLY, 0, 1, 0),
   F_B(HIS_WALLOPS, 0, 1, 0),
   F_B(HIS_MAP, 0, 1, 0),
   F_B(HIS_LINKS, 0, 1, 0),
   F_B(HIS_TRACE, 0, 1, 0),
-  F_B(HIS_STATS_l, 0, 1, 0),
+  F_B(HIS_STATS_a, 0, 1, 0),
   F_B(HIS_STATS_c, 0, 1, 0),
-  F_B(HIS_STATS_g, 0, 1, 0),
-  F_B(HIS_STATS_h, 0, 1, 0),
-  F_B(HIS_STATS_k, 0, 1, 0),
+  F_B(HIS_STATS_d, 0, 1, 0),
+  F_B(HIS_STATS_e, 0, 1, 0),
   F_B(HIS_STATS_f, 0, 1, 0),
+  F_B(HIS_STATS_g, 0, 1, 0),
   F_B(HIS_STATS_i, 0, 1, 0),
   F_B(HIS_STATS_j, 0, 1, 0),
+  F_B(HIS_STATS_J, 0, 1, 0),
+  F_B(HIS_STATS_k, 0, 1, 0),
+  F_B(HIS_STATS_l, 0, 1, 0),
+  F_B(HIS_STATS_L, 0, 1, 0),
   F_B(HIS_STATS_M, 0, 1, 0),
   F_B(HIS_STATS_m, 0, 1, 0),
   F_B(HIS_STATS_o, 0, 1, 0),
@@ -323,8 +392,6 @@ static struct FeatureDesc {
   F_B(HIS_STATS_q, 0, 1, 0),
   F_B(HIS_STATS_R, 0, 1, 0),
   F_B(HIS_STATS_r, 0, 1, 0),
-  F_B(HIS_STATS_d, 0, 1, 0),
-  F_B(HIS_STATS_e, 0, 1, 0),
   F_B(HIS_STATS_t, 0, 1, 0),
   F_B(HIS_STATS_T, 0, 1, 0),
   F_B(HIS_STATS_u, 0, 0, 0),
@@ -334,22 +401,26 @@ static struct FeatureDesc {
   F_B(HIS_STATS_x, 0, 1, 0),
   F_B(HIS_STATS_y, 0, 1, 0),
   F_B(HIS_STATS_z, 0, 1, 0),
+  F_B(HIS_STATS_IAUTH, 0, 1, 0),
   F_B(HIS_WHOIS_SERVERNAME, 0, 1, 0),
   F_B(HIS_WHOIS_IDLETIME, 0, 1, 0),
+  F_B(HIS_WHOIS_LOCALCHAN, 0, 1, 0),
   F_B(HIS_WHO_SERVERNAME, 0, 1, 0),
   F_B(HIS_WHO_HOPCOUNT, 0, 1, 0),
+  F_B(HIS_MODEWHO, 0, 1, 0),
   F_B(HIS_BANWHO, 0, 1, 0),
   F_B(HIS_KILLWHO, 0, 1, 0),
   F_B(HIS_REWRITE, 0, 1, 0),
   F_I(HIS_REMOTE, 0, 1, 0),
   F_B(HIS_NETSPLIT, 0, 1, 0),
-  F_S(HIS_SERVERNAME, 0, "*.undernet.org", 0),
-  F_S(HIS_SERVERINFO, 0, "The Undernet Underworld", 0),
+  F_S(HIS_SERVERNAME, 0, "*.undernet.org", feature_notify_servername),
+  F_S(HIS_SERVERINFO, 0, "The Undernet Underworld", feature_notify_serverinfo),
   F_S(HIS_URLSERVERS, 0, "http://www.undernet.org/servers.php", 0),
 
   /* Misc. random stuff */
   F_S(NETWORK, 0, "UnderNet", 0),
   F_S(URL_CLIENTS, 0, "ftp://ftp.undernet.org/pub/irc/clients", 0),
+  F_S(URLREG, 0, "http://cservice.undernet.org/live/", 0),
 
 #undef F_S
 #undef F_B
@@ -358,7 +429,11 @@ static struct FeatureDesc {
   { FEAT_LAST_F, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0 }
 };
 
-/* Given a feature's identifier, look up the feature descriptor */
+/** Given a feature's identifier, look up the feature descriptor.
+ * @param[in] from Client looking up feature, or NULL.
+ * @param[in] feature Feature name to find.
+ * @return Pointer to a FeatureDesc, or NULL if none was found.
+ */
 static struct FeatureDesc *
 feature_desc(struct Client* from, const char *feature)
 {
@@ -379,7 +454,12 @@ feature_desc(struct Client* from, const char *feature)
   return 0; /* not found */
 }
 
-/* Given a feature vector string, set the value of a feature */
+/** Given a feature vector string, set the value of a feature.
+ * @param[in] from Client trying to set the feature, or NULL.
+ * @param[in] fields Parameters to set, starting with feature name.
+ * @param[in] count Number of fields in \a fields.
+ * @return Zero (or, theoretically, CPTR_KILLED).
+ */
 int
 feature_set(struct Client* from, const char* const* fields, int count)
 {
@@ -522,12 +602,20 @@ feature_set(struct Client* from, const char* const* fields, int count)
 
     if (change && feat->notify) /* call change notify function */
       (*feat->notify)();
+
+    if (from)
+      return feature_get(from, fields, count);
   }
 
   return 0;
 }
 
-/* reset a feature to its default values */
+/** Reset a feature to its default values.
+ * @param[in] from Client trying to reset the feature, or NULL.
+ * @param[in] fields Parameters to set, starting with feature name.
+ * @param[in] count Number of fields in \a fields.
+ * @return Zero (or, theoretically, CPTR_KILLED).
+ */
 int
 feature_reset(struct Client* from, const char* const* fields, int count)
 {
@@ -580,12 +668,20 @@ feature_reset(struct Client* from, const char* const* fields, int count)
 
     if (change && feat->notify) /* call change notify function */
       (*feat->notify)();
+
+    if (from)
+      return feature_get(from, fields, count);
   }
 
   return 0;
 }
 
-/* Gets the value of a specific feature and reports it to the user */
+/** Gets the value of a specific feature and reports it to the user.
+ * @param[in] from Client trying to get the feature.
+ * @param[in] fields Parameters to set, starting with feature name.
+ * @param[in] count Number of fields in \a fields.
+ * @return Zero (or, theoretically, CPTR_KILLED).
+ */
 int
 feature_get(struct Client* from, const char* const* fields, int count)
 {
@@ -632,7 +728,7 @@ feature_get(struct Client* from, const char* const* fields, int count)
   return 0;
 }
 
-/* called before reading the .conf to clear all marks */
+/** Called before reading the .conf to clear all dirty marks. */
 void
 feature_unmark(void)
 {
@@ -645,7 +741,7 @@ feature_unmark(void)
   }
 }
 
-/* Called after reading the .conf to reset unmodified values to defaults */
+/** Called after reading the .conf to reset unmodified values to defaults. */
 void
 feature_mark(void)
 {
@@ -687,7 +783,7 @@ feature_mark(void)
   }
 }
 
-/* used to initialize the features subsystem */
+/** Initialize the features subsystem. */
 void
 feature_init(void)
 {
@@ -709,12 +805,23 @@ feature_init(void)
       break;
     }
   }
+
+  cli_magic(&his) = CLIENT_MAGIC;
+  cli_status(&his) = STAT_SERVER;
+  feature_notify_servername();
+  feature_notify_serverinfo();
 }
 
-/* report all F-lines */
+/** Report all F-lines to a user.
+ * @param[in] to Client requesting statistics.
+ * @param[in] sd Stats descriptor for request (ignored).
+ * @param[in] param Extra parameter from user (ignored).
+ */
 void
-feature_report(struct Client* to, struct StatDesc* sd, int stat, char* param)
+feature_report(struct Client* to, const struct StatDesc* sd, char* param)
 {
+  char changed;
+  int report;
   int i;
 
   for (i = 0; features[i].type; i++) {
@@ -723,40 +830,46 @@ feature_report(struct Client* to, struct StatDesc* sd, int stat, char* param)
        (features[i].flags & FEAT_OPER && !IsAnOper(to)))
       continue; /* skip this one */
 
+    changed = (features[i].flags & FEAT_MARK) ? 'F' : 'f';
+    report = (features[i].flags & FEAT_MARK) || sd->sd_funcdata;
+
     switch (features[i].flags & FEAT_MASK) {
     case FEAT_NONE:
       if (features[i].report) /* let the callback handle this */
-       (*features[i].report)(to, features[i].flags & FEAT_MARK ? 1 : 0);
+       (*features[i].report)(to, report);
       break;
 
 
     case FEAT_INT: /* Report an F-line with integer values */
-      if (features[i].flags & FEAT_MARK) /* it's been changed */
-       send_reply(to, SND_EXPLICIT | RPL_STATSFLINE, "F %s %d",
-                  features[i].type, features[i].v_int);
+      if (report) /* it's been changed */
+       send_reply(to, SND_EXPLICIT | RPL_STATSFLINE, "%c %s %d",
+                  changed, features[i].type, features[i].v_int);
       break;
 
     case FEAT_BOOL: /* Report an F-line with boolean values */
-      if (features[i].flags & FEAT_MARK) /* it's been changed */
-       send_reply(to, SND_EXPLICIT | RPL_STATSFLINE, "F %s %s",
-                  features[i].type, features[i].v_int ? "TRUE" : "FALSE");
+      if (report) /* it's been changed */
+       send_reply(to, SND_EXPLICIT | RPL_STATSFLINE, "%c %s %s",
+                  changed, features[i].type, features[i].v_int ? "TRUE" : "FALSE");
       break;
 
     case FEAT_STR: /* Report an F-line with string values */
-      if (features[i].flags & FEAT_MARK) { /* it's been changed */
+      if (report) { /* it's been changed */
        if (features[i].v_str)
-         send_reply(to, SND_EXPLICIT | RPL_STATSFLINE, "F %s %s",
-                    features[i].type, features[i].v_str);
+         send_reply(to, SND_EXPLICIT | RPL_STATSFLINE, "%c %s %s",
+                    changed, features[i].type, features[i].v_str);
        else /* Actually, F:<type> would reset it; you want F:<type>: */
-         send_reply(to, SND_EXPLICIT | RPL_STATSFLINE, "F %s",
-                    features[i].type);
+         send_reply(to, SND_EXPLICIT | RPL_STATSFLINE, "%c %s",
+                    changed, features[i].type);
       }
       break;
     }
   }
 }
 
-/* return a feature's integer value */
+/** Return a feature's integer value.
+ * @param[in] feat &Feature identifier.
+ * @return Integer value of feature.
+ */
 int
 feature_int(enum Feature feat)
 {
@@ -766,7 +879,10 @@ feature_int(enum Feature feat)
   return features[feat].v_int;
 }
 
-/* return a feature's boolean value */
+/** Return a feature's boolean value.
+ * @param[in] feat &Feature identifier.
+ * @return Boolean value of feature.
+ */
 int
 feature_bool(enum Feature feat)
 {
@@ -779,7 +895,10 @@ feature_bool(enum Feature feat)
   return features[feat].v_int;
 }
 
-/* return a feature's string value */
+/** Return a feature's string value.
+ * @param[in] feat &Feature identifier.
+ * @return String value of feature.
+ */
 const char *
 feature_str(enum Feature feat)
 {