Atoms: Avoid allocations in XkbAtomText()

XkbAtomGetString() returns a freshly-allocated string, whereas
XkbAtomText() returns the same in a temporary buffer.  XkbAtomText used
to call XkbAtomGetString() and then free the result, which seems quite
spectacularly pointless when you think about it.  Shuffle the atom code
around so we don't have to allocate for XkbAtomText().

This changes semantics slightly wrt non-printable characters, but I
haven't been able to see any effect so far.  And it may well be ever so
slightly quicker.

Signed-off-by: Daniel Stone <daniel@fooishbar.org>

1 files changed, 27 insertions, 3 deletions

diff --git a/src/atom.c b/src/atom.c
index 127716f..9ca307c 100644
--- a/src/atom.c
+++ b/src/atom.c
@@ -104,8 +104,8 @@ XkbcInitAtoms(void)
     nodeTable[None] = NULL;
 }
 
-char *
-XkbcAtomGetString(Atom atom)
+static char *
+_XkbcAtomGetString(Atom atom)
 {
     NodePtr node;
 
@@ -113,7 +113,31 @@ XkbcAtomGetString(Atom atom)
         return NULL;
     if (!(node = nodeTable[atom]))
         return NULL;
-    return strdup(node->string);
+    return node->string;
+}
+
+char *
+XkbcAtomGetString(Atom atom)
+{
+    char *ret = _XkbcAtomGetString(atom);
+    return ret ? strdup(ret) : NULL;
+}
+
+char *
+XkbcAtomText(Atom atom)
+{
+    char *tmp, *ret;
+
+    tmp = _XkbcAtomGetString(atom);
+    if (!tmp)
+        return "";
+
+    ret = tbGetBuffer(strlen(tmp) + 1);
+    if (!ret)
+        return "";
+
+    strcpy(ret, tmp);
+    return ret;
 }
 
 static Atom