Changeset 6681 for trunk/ARBDB/arbdb.cxx

Timestamp:

30/07/10 16:11:07 (19 months ago)

Author:

westram

Message:

• added GB_key_2_existing_quark (like GB_key_2_quark, but returns 0 for non-existing quarks)
• use GB_key_2_existing_quark in gb_delete_main_entry (avoid creating GB_SYSTEM_FOLDER after failed client connect). This fixes crash of arb_message when no server was running! (introduced in [6648])
• gb_pre_delete_entry
  • if no remote_hash exists, do not erase anything from there
• rewrote GB_key_2_quark() and gb_key_2_quark() using gb_key_2_existing_quark()

Files:

• trunk/ARBDB/arbdb.cxx (modified) (2 diffs)

trunk/ARBDB/arbdb.cxx

@@ -1244 +1244 @@
 
 
-GBQUARK GB_key_2_quark(GBDATA *gbd, const char *s) {
-    long          index;
-    GB_MAIN_TYPE *Main = GB_MAIN(gbd);
-
-    if (!s) return -1;
-    index = GBS_read_hash(Main->key_2_index_hash, s);
-    if (!index) {   // create new index
-        index = gb_create_key(Main, s, true);
-    }
-    return (GBQUARK)index;
-}
+GBQUARK gb_key_2_existing_quark(GB_MAIN_TYPE *Main, const char *key) {
+    /*! @return existing quark for 'key'
+     *          -1 = key is NULL
+     *           0 = no quark exists
+     *        else = quark
+     */
+
+    GBQUARK quark  = -1;
+    if (key) quark = GBS_read_hash(Main->key_2_index_hash, key);
+    return quark;
+}
+
+GBQUARK GB_key_2_quark(GBDATA *gbd, const char *key) {
+    /*! find or create quark for 'key'
+     * @return -1 if key is NULL, else existing or created quark
+     */
+
+    GB_MAIN_TYPE *Main  = GB_MAIN(gbd);
+    GBQUARK       quark = gb_key_2_existing_quark(Main, key);
+    if (!quark) quark   = gb_create_key(GB_MAIN(gbd), key, true);
+    return quark;
+}
+
+#if defined(DEVEL_RALF)
+#warning add gb_NULLkey_2_quark allowing NULL as key
+#endif // DEVEL_RALF
+
+GBQUARK gb_key_2_quark(GB_MAIN_TYPE *Main, const char *key) {
+    // similar to GB_key_2_quark, 
+    // but if 'key' is NULL, quark 0 will be returned.
+    // 
+    // Use this function with care.
+    //
+    // Known good use:
+    // - create main entry and its dummy father via gb_make_container()
+    //
+    // Other uses (maybe just to pass GB_MAIN_TYPE instead of GBDATA as GB_key_2_quark likes):
+    // - gb_rename_entry
+    // - gb_make_entry
+    // - compress_sequence_tree
+    // - dictionary compression (several calls)
+
+    GBQUARK quark = 0;
+    if (key) {
+        quark             = gb_key_2_existing_quark(Main, key);
+        if (!quark) quark = gb_create_key(Main, key, true);
+        gb_assert(quark>0);
+    }
+    return quark;
+}
+
+// ---------------------------------------------
 
 GBQUARK GB_get_quark(GBDATA *gbd) {
@@ -1265 +1306 @@
 }
 
-GBQUARK gb_key_2_quark(GB_MAIN_TYPE *Main, const char *s) {
-    long index;
-    if (!s) return 0;
-    index = GBS_read_hash(Main->key_2_index_hash, s);
-    if (!index) {   // create new index
-        index = gb_create_key(Main, s, true);
-    }
-    return (GBQUARK)index;
-}
-
-
-
-
-long GB_read_clock(GBDATA *gbd)
-{
+// ---------------------------------------------
+
+long GB_read_clock(GBDATA *gbd) {
     if (GB_ARRAY_FLAGS(gbd).changed) return GB_MAIN(gbd)->clock;
     return GB_GET_EXT_UPDATE_DATE(gbd);
 }
 
-long    GB_read_transaction(GBDATA *gbd)
-{
+long GB_read_transaction(GBDATA *gbd) {
     return GB_MAIN(gbd)->transaction;
 }