Changeset 1257

Timestamp:

2000-06-30T12:40:39+12:00 (24 years ago)

Author:

sjboddie

Message:

Tidied up a bit. Fixed bug in formattools. Nested If/Or's should now
work within formatstrings.

File:

• trunk/gsdl/src/recpt/formattools.cpp (modified) (6 diffs)

trunk/gsdl/src/recpt/formattools.cpp

@@ -28 +28 @@
 /*
    $Log$
+   Revision 1.21  2000/06/30 00:40:39  sjboddie
+   Tidied up a bit. Fixed bug in formattools. Nested If/Or's should now
+   work within formatstrings.
+
    Revision 1.20  2000/04/07 04:40:44  sjboddie
    Reverted back to old DocumentHeader, DocumentTitles, DocumentImages etc.
@@ -98 +102 @@
 #include "formattools.h"
 #include "cgiutils.h"
+#include <assert.h>
 
 // a few function prototypes
@@ -153 +158 @@
   while (here != end) {
     if (*here != ' ') {
-      if (*here == '<') {
+      if ((*here == '<') && ((here+3) < end)) {
     if ((*(here+1) == 't' || *(here+1) == 'T') &&
         (*(here+2) == 'd' || *(here+2) == 'D') &&
@@ -239 +244 @@
 
 static void get_parent_options (text_t &instring, metadata_t &metaoption) {
+
+  assert (instring.size() > 7);
+  if (instring.size() <= 7) return;
 
   text_t meta, com, op;
@@ -323 +331 @@
   while (here != end) {
 
-    if (*here == '\\')
-      text.push_back (*(++here));
-
-    else if (*here == '{') {
+    if (*here == '\\') {
+      here ++;
+      if (here != end) text.push_back (*here);
+
+    } else if (*here == '{') {
       if (!text.empty()) {
     formatlistptr->command = comText;
@@ -433 +442 @@
 
     } else {
-      // assume it's plain text
-      or_ptr->command = comText;
-      or_ptr->text = text;
+      parse_string (text, or_ptr, metadata, getParents);
     }
     text.clear();