Changeset 6811 for trunk/ARBDB/ad_config.cxx

Timestamp:

09/09/10 13:03:26 (21 months ago)

Author:

westram

Message:

• report more information about broken configurations
• GBT_commit_rename_session did NEVER report error (since [2])
  • added __ATTR__USERESULT
• forward error in AWTC_pars_names (and abort complete rename)
• this fixes the bug with noticable broken configs after merge/rename
  • cause of bug: the config parser used inside ABRDB is much more picky than the one used in NTREE (needs to be DRYed)
  • no idea when this bug was introduced (maybe by introduction of GBT_config_parser in [3824]) and why it popped up now

Files:

• trunk/ARBDB/ad_config.cxx (modified) (1 diff)

trunk/ARBDB/ad_config.cxx

@@ -195 +195 @@
         }
 
-        switch (item->type) {
-            case CI_CLOSE_GROUP:
-                pos += 2;
-                break;
-
-            case CI_UNKNOWN:
-                error = GBS_global_string_copy("Unknown flag '%c'", label);
-                break;
-
-            default: {
-                const char *start_of_name = str+pos+2;
-                const char *behind_name   = strchr(start_of_name, '\1');
-
-                if (!behind_name) behind_name = strchr(start_of_name, '\0'); // eos
-                gb_assert(behind_name);
-
-                item->name = (char*)GB_calloc(1, behind_name-start_of_name+1);
-                memcpy(item->name, start_of_name, behind_name-start_of_name);
-                pos       = behind_name-str;
-                break;
+        if (item->type == CI_CLOSE_GROUP) {
+            pos += 2;
+        }
+        else {
+            const char *start_of_name = str+pos+2;
+            const char *behind_name   = strchr(start_of_name, '\1');
+
+            if (!behind_name) behind_name = strchr(start_of_name, '\0'); // eos
+            gb_assert(behind_name);
+
+            char *data = GB_strpartdup(start_of_name, behind_name-1);
+            if (item->type == CI_UNKNOWN) {
+                error = GBS_global_string_copy("Unknown flag '%c' (followed by '%s')", label, data);
+                free(data);
+            }
+            else {
+                item->name = data;
+                pos        = behind_name-str;
             }
         }