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

Comparing rxvt-unicode/src/xdefaults.C (file contents):
Revision 1.191 by sf-exg, Thu Jul 14 05:10:06 2016 UTC vs.
Revision 1.195 by sf-exg, Tue May 25 05:11:51 2021 UTC

               STRG (Rs_scrollBar_thickness, "thickness", "sbt", "number", "scrollbar thickness/width in pixels"),
               BOOL (Rs_scrollTtyOutput, "scrollTtyOutput", NULL, Opt_scrollTtyOutput, 0, NULL),
               BOOL (Rs_scrollTtyOutput, NULL, "si",  Opt_scrollTtyOutput, Optflag_Reverse, "scroll-on-tty-output inhibit"),
               BOOL (Rs_scrollTtyKeypress, "scrollTtyKeypress", "sk", Opt_scrollTtyKeypress, 0, "scroll-on-keypress"),
               BOOL (Rs_scrollWithBuffer, "scrollWithBuffer", "sw", Opt_scrollWithBuffer, 0, "scroll-with-buffer"),
-#if BG_IMAGE_FROM_ROOT
-              BOOL (Rs_transparent, "inheritPixmap", "ip", Opt_transparent, 0, "inherit parent pixmap"),
-              BOOL (Rs_transparent, "transparent", "tr", Opt_transparent, 0, "inherit parent pixmap"),
-              STRG (Rs_color + Color_tint, "tintColor", "tint", "color", "tint color"),
-              STRG (Rs_shade, "shading", "sh", "number", "shade background by number %."),
-              STRG (Rs_blurradius, "blurRadius", "blr", "HxV", "gaussian blur radii to apply to the root background"),
-#endif
 #if OFF_FOCUS_FADING
               STRG (Rs_fade, "fading", "fade", "number", "fade colors by number % when losing focus"),
               STRG (Rs_color + Color_fade, "fadeColor", "fadecolor", "color", "target color for off-focus fading"),
 #endif
               BOOL (Rs_utmpInhibit, "utmpInhibit", "ut", Opt_utmpInhibit, 0, "utmp inhibit"),
               RSTRG (Rs_color + Color_cursor2, "cursorColor2", "color"),
 #endif /* NO_CURSORCOLOR */
               STRG (Rs_color + Color_pointer_fg, "pointerColor", "pr", "color", "pointer color"),
               STRG (Rs_color + Color_pointer_bg, "pointerColor2", "pr2", "color", "pointer bg color"),
               STRG (Rs_color + Color_border, "borderColor", "bd", "color", "border color"),
-#if BG_IMAGE_FROM_FILE
-              RSTRG (Rs_path, "path", "search path"),
-              STRG (Rs_backgroundPixmap, "backgroundPixmap", "pixmap", "file[;geom]", "background pixmap"),
-#endif
 #if ENABLE_EWMH
               STRG (Rs_iconfile, "iconFile", "icon", "file", "path to application icon image"),
 #endif
 #ifdef HAVE_XMU
               RSTRG (Rs_pointerShape, "pointerShape", "string"),
 #ifdef KEYSYM_RESOURCE
 static void
 rxvt_define_key (rxvt_term *term, const char *k, const char *v)
 {
   term->bind_action (k, v);
-}
-/*
- * Define key from XrmEnumerateDatabase.
- *   quarks will be something like
- *      "rxvt" "keysym" "0xFF01"
- *   value will be a string
- */
-static int
-rxvt_enumerate_helper (
-   XrmDatabase *database ecb_unused,
-   XrmBindingList bindings ecb_unused,
-   XrmQuarkList quarks,
-   XrmRepresentation *type ecb_unused,
-   XrmValue *value,
-   XPointer closure
-)
-{
-  int last;
-  for (last = 0; quarks[last] != NULLQUARK; last++)	/* look for last quark in list */
-    ;
-  rxvt_term *term = (rxvt_term *)(((void **)closure)[0]);
-  void (*cb)(rxvt_term *, const char *, const char *)
-     = (void (*)(rxvt_term *, const char *, const char *))
-          (((void **)closure)[1]);
-  cb (term, XrmQuarkToString (quarks[last - 1]), (char *)value->addr);
-  return False;
 }
 /*
  * look for something like this (XK_Delete)
  * rxvt*keysym.0xFFFF: "\177"
         }
     }
 #endif /* NO_RESOURCES */
 }
+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
+ */
+static int
+rxvt_enumerate_helper (
+   XrmDatabase *database ecb_unused,
+   XrmBindingList bindings ecb_unused,
+   XrmQuarkList quarks,
+   XrmRepresentation *type ecb_unused,
+   XrmValue *value,
+   XPointer closure
+)
+{
+  const rxvt_enumerate_closure *data = (const rxvt_enumerate_closure *)closure;
+  if (*quarks == NULLQUARK) return False;
+  // if the quark list starts with a tightly bound quark, we skip it,
+  // as it exactly matched the prefix. Otherwise, it matched because
+  // it started with "*", in which case we assume 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;
+    }
+  // specific, a bit misleadingly named, is used when a specific "subclass"
+  // is iterated over, e.g. "keysym", and is used to skip one more
+  // component, as well as all generic prefixes
+  // 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;
+      ++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, fast path
+  else
+    {
+      // multiple components, slow path - should be rare, 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;
+}
 void
 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.191 by sf-exg, Thu Jul 14 05:10:06 2016 UTC vs. Revision 1.195 by sf-exg, Tue May 25 05:11:51 2021 UTC

Diff Legend

Comparing rxvt-unicode/src/xdefaults.C (file contents):
Revision 1.191 by sf-exg, Thu Jul 14 05:10:06 2016 UTC vs.
Revision 1.195 by sf-exg, Tue May 25 05:11:51 2021 UTC