[ViewVC] Diff of: cvs/rxvt-unicode/src/xdefaults.C

Comparing rxvt-unicode/src/xdefaults.C (file contents):
Revision 1.192 by sf-exg, Thu Jul 14 05:33:26 2016 UTC vs.
Revision 1.193 by root, Tue Sep 17 20:38:14 2019 UTC

 rxvt_define_key (rxvt_term *term, const char *k, const char *v)
 {
   term->bind_action (k, v);
 }
+struct rxvt_enumerate_closure
+{
+  rxvt_term *term;
+  void (*cb)(rxvt_term *, const char *, const char *);
+  int specific; // iterate over only a specific subhierarchy
+};
 /*
  * Define key from XrmEnumerateDatabase.
  *   quarks will be something like
  *      "rxvt" "keysym" "0xFF01"
  *   value will be a string
    XrmRepresentation *type ecb_unused,
    XrmValue *value,
    XPointer closure
 )
 {
-  int last;
+  const rxvt_enumerate_closure *data = (const rxvt_enumerate_closure *)closure;
-  for (last = 0; quarks[last] != NULLQUARK; last++)	/* look for last quark in list */
+  if (*quarks == NULLQUARK) return False;
+  // if the quark list starts with a tighly bound quark, we skip it,
+  // as it is the exactly matched the prefix. Otherwise, it matched because
+  // it started with "*", in which case we assuime the prefix is part
+  // of the "*".
+  if (*bindings == XrmBindTightly)
-    ;
+    {
+      ++quarks, ++bindings; // skip if this is a fixed prefix, rather than a *-match
+      if (*quarks == NULLQUARK) return False;
+    }
-  rxvt_term *term = (rxvt_term *)(((void **)closure)[0]);
+  // specific, a bit misleadingly named, is used when a specific "subclass"
-  void (*cb)(rxvt_term *, const char *, const char *)
+  // is iterated over, e.g. "keysym", and is used to skip one more
-     = (void (*)(rxvt_term *, const char *, const char *))
+  // component, as well as all generic prefixes
-          (((void **)closure)[1]);
+  // this is a bit of a hack, ideally, keysym (the only user) should use its
+  // own iteration function, but this ought to be less bloated
+  if (data->specific)
+    {
+      if (*bindings != XrmBindTightly)
+        return False;
-  cb (term, XrmQuarkToString (quarks[last - 1]), (char *)value->addr);
+      ++quarks, ++bindings; // skip if this is a fixed prefix, rather than a *-match
+      if (*quarks == NULLQUARK) return False;
+    }
+  char *pattern;
+  if (quarks[1] == NULLQUARK)
+    pattern = XrmQuarkToString (quarks[0]); // single component, fats path
+  else
+    {
+      // multiple components, slow path - should be rare, to don't optimize for speed
+      int size = 0;
+      for (int i = 0; quarks[i] != NULLQUARK; ++i)
+        size += strlen (XrmQuarkToString (quarks[i])) + 1;
+      pattern = rxvt_temp_buf<char> (size + 1);
+      // now print all components
+      {
+        char *cur = pattern;
+        for (int i = 0; quarks[i] != NULLQUARK; ++i)
+          cur += sprintf (cur, ".%s", XrmQuarkToString (quarks[i]));
+      }
+      ++pattern; // skip initial dot
+    }
+  data->cb (data->term, pattern, (char *)value->addr);
   return False;
 }
 /*
 rxvt_term::enumerate_resources (void (*cb)(rxvt_term *, const char *, const char *), const char *name_p, const char *class_p)
 {
   /*
    * [R5 or later]: enumerate the resource database
    */
+  assert (!name_p == !class_p); // both must be specified, or missing
 #ifdef KEYSYM_RESOURCE
-  void *closure[2] = {
+  rxvt_enumerate_closure closure = { this, cb, name_p ? 1 : 0 };
-    (void *)this,
-    (void *)cb,
-  };
   XrmDatabase database = XrmGetDatabase (dpy);
   XrmName name_prefix[3];
   XrmClass class_prefix[3];
   class_prefix[1] = class_p ? XrmStringToName (class_p) : NULLQUARK;
   class_prefix[2] = NULLQUARK;
 # ifdef RESFALLBACK
   name_prefix[0] = class_prefix[0] = XrmStringToName (RESFALLBACK);
-  /* XXX: Need to check sizeof (rxvt_t) == sizeof (XPointer) */
   XrmEnumerateDatabase (database, name_prefix, class_prefix,
-                        XrmEnumOneLevel, rxvt_enumerate_helper, (XPointer)closure);
+                        XrmEnumAllLevels, rxvt_enumerate_helper, (XPointer)&closure);
 # endif
   name_prefix[0] = class_prefix[0] = XrmStringToName (RESCLASS);
   XrmEnumerateDatabase (database, name_prefix, class_prefix,
-                        XrmEnumOneLevel, rxvt_enumerate_helper, (XPointer)closure);
+                        XrmEnumAllLevels, rxvt_enumerate_helper, (XPointer)&closure);
   name_prefix[0] = class_prefix[0] = XrmStringToName (rs[Rs_name]);
   XrmEnumerateDatabase (database, name_prefix, class_prefix,
-                        XrmEnumOneLevel, rxvt_enumerate_helper, (XPointer)closure);
+                        XrmEnumAllLevels, rxvt_enumerate_helper, (XPointer)&closure);
 #endif
 }
 void
 rxvt_term::extract_keysym_resources ()

Diff Legend

-–
+Removed lines
-+
+Added lines
-<
+Changed lines
->
+Changed lines

Comparing rxvt-unicode/src/xdefaults.C (file contents): Revision 1.192 by sf-exg, Thu Jul 14 05:33:26 2016 UTC vs. Revision 1.193 by root, Tue Sep 17 20:38:14 2019 UTC

Diff Legend

Comparing rxvt-unicode/src/xdefaults.C (file contents):
Revision 1.192 by sf-exg, Thu Jul 14 05:33:26 2016 UTC vs.
Revision 1.193 by root, Tue Sep 17 20:38:14 2019 UTC