From fbe7e0224704f0ab1dff0a438323d8519ecca75e Mon Sep 17 00:00:00 2001
From: onefang
Date: Thu, 7 May 2020 11:16:04 +1000
Subject: Various comment tweaks.

---
 src/sledjchisl/sledjchisl.c | 21 ++++++++++++++++++---
 1 file changed, 18 insertions(+), 3 deletions(-)

(limited to 'src/sledjchisl/sledjchisl.c')

diff --git a/src/sledjchisl/sledjchisl.c b/src/sledjchisl/sledjchisl.c
index c1f6f13..55e70f7 100644
--- a/src/sledjchisl/sledjchisl.c
+++ b/src/sledjchisl/sledjchisl.c
@@ -4920,6 +4920,7 @@ static void accountEditWeb(reqData *Rd, inputForm *oF, inputValue *oV)
     HTMLhidden(Rd->reply, "user", name);
     Rd->reply->addstrf(Rd->reply, "<p><font size='5'><span style='font-size: x-large'><b>Name :</b></span></font> %s</p>", name);
     Rd->reply->addstrf(Rd->reply, "<p><font size='5'><span style='font-size: x-large'><b>Email :</b></span></font> %s</p>", email);
+// TODO - add aboutMe here in read only mode.
     Rd->reply->addstrf(Rd->reply, "<p><font size='5'><span style='font-size: x-large'><b>Voucher :</b></span></font> %s</p>", voucher);
 
     if (200 <= Rd->shs.level)
@@ -5164,6 +5165,7 @@ static int accountDelSub(reqData *Rd, inputForm *iF, inputValue *iV)
   return ret;
 }
 
+// The [create member] button on accountLoginWeb()
 static int accountCreateSub(reqData *Rd, inputForm *iF, inputValue *iV)
 {
   int ret = 0;
@@ -5200,6 +5202,7 @@ static int accountCreateSub(reqData *Rd, inputForm *iF, inputValue *iV)
   return ret;
 }
 
+// The [confirm] button on accountAddWeb()
 static int accountAddSub(reqData *Rd, inputForm *iF, inputValue *iV)
 {
   int ret = 0;
@@ -5290,6 +5293,7 @@ static int accountSaveSub(reqData *Rd, inputForm *iF, inputValue *iV)
   return ret;
 }
 
+// The unique validation URL sent in email.
 static int accountValidateSub(reqData *Rd, inputForm *iF, inputValue *iV)
 {
   int ret = 0;
@@ -5411,6 +5415,17 @@ static int accountOutSub(reqData *Rd, inputForm *iF, inputValue *iV)
   return ret;
 }
 
+/* TODO - instead of searching through all the users, ...
+  have a bunch of separate folders with symlinks
+  scData/users/aaproved
+  scData/users/disabled
+  scData/users/god
+    onefang_rejected.lua	-> ../uuid.lua
+  scData/users/newbie
+    foo_bar.lua			-> ../uuid.lua
+  scData/users/validated
+
+*/
 typedef struct _RdAndListTbl RdAndListTbl;
 struct _RdAndListTbl
 {
@@ -6138,6 +6153,7 @@ void account_html(char *file, reqData *Rd, HTMLfile *thisFile)
     else
     {
       // Redirect to a GET if it was a POST.
+      // The reason here is to avoid a refresh turning into a rePOST.
       if ((strcmp("POST", Rd->Method) == 0))
       {
 	if ('\0' != Rd->form[0])
@@ -6161,7 +6177,7 @@ void account_html(char *file, reqData *Rd, HTMLfile *thisFile)
       E("Building output ERROR page %s, coz of %d errors.", iF->name, e);
     else
       D("Building alternate output page %s", iF->name);
-    // First just sort out groups, then get the data from Rd->stuff.
+    // First just sort out input groups, then get the data from Rd->stuff.
     count = iF->fields->size(iF->fields);
     iV = xzalloc(count * sizeof(inputValue));
     collectFields(Rd, iF, iV, -1);
@@ -6536,7 +6552,7 @@ jit library is loaded or the JIT compiler will not be activated.
 // TODO - it looks like OpenSim invented their own half arsed backwards INI file include system.
 //          I doubt qLibc supports it, like it supports what seems to be the standard include system.
 //	    Not sure if we need to worry about it just yet.
-// TODO - this leaks memory, but I suspect it's a bug in qLibc.
+// TODO - this leaks memory, but it's a bug in qLibc.  Send the bug fix upstream.
     qlisttbl_t *qconfig = qconfig_parse_file(NULL, toybuf, '=');
     if (NULL == qconfig)
     {
@@ -7045,7 +7061,6 @@ fcgiDone:
     }
 
     for (i = 0; i < sims->num; i++)
-//    for (i = 0; i < 2; i++)
     {
       char *sim = sims->sims[i], *name = getSimName(sims->sims[i]);
 
-- 
cgit v1.1