From 36afdebe1ac08fe02c938a73270bd2f11999d677 Mon Sep 17 00:00:00 2001
From: Robert Ransom <rransom.8774@gmail.com>
Date: Wed, 18 May 2011 04:33:48 -0700
Subject: [PATCH] Add an XXX

---
 src/or/control.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/src/or/control.c b/src/or/control.c
index f7ff92ab34..47e7081825 100644
--- a/src/or/control.c
+++ b/src/or/control.c
@@ -2894,6 +2894,9 @@ connection_control_process_inbuf(control_connection_t *conn)
     return 0;
   }
 
+  /* XXXX Why is this not implemented as a table like the GETINFO
+   * items are?  Even handling the plus signs at the beginnings of
+   * commands wouldn't be very hard with proper macros. */
   cmd_data_len = (uint32_t)data_len;
   if (!strcasecmp(conn->incoming_cmd, "SETCONF")) {
     if (handle_control_setconf(conn, cmd_data_len, args))