better string get/set

[chaz/openbox] / src / XAtom.cc

diff --git a/src/XAtom.cc b/src/XAtom.cc
index 0f36ef9b86fd7aa45732a70ae2334ae5dcbb6170..9e5901265f0b5953c93e7719a45d5bdeecea3ec6 100644 (file)
--- a/src/XAtom.cc
+++ b/src/XAtom.cc
@@ -182,7 +182,8 @@ void XAtom::setPixmapValue(Window win, Atom atom, Pixmap value) const {
 /*
  * Set a string property value on a window.
  */
-void XAtom::setStringValue(Window win, Atom atom, std::string &value) const {
+void XAtom::setStringValue(Window win, Atom atom,
+                           const std::string &value) const {
   setValue(win, atom, XA_STRING,
            const_cast<unsigned char*>
            (reinterpret_cast<const unsigned char*>(value.c_str())),
@@ -229,7 +230,8 @@ void XAtom::addPixmapValue(Window win, Atom atom, Pixmap value) const {
 /*
  * Add characters to a string property value on a window.
  */
-void XAtom::addStringValue(Window win, Atom atom, std::string &value) const {
+void XAtom::addStringValue(Window win, Atom atom,
+                           const std::string &value) const {
   setValue(win, atom, XA_STRING,
            const_cast<unsigned char*>
            (reinterpret_cast<const unsigned char *>
@@ -328,10 +330,10 @@ bool XAtom::getPixmapValue(Window win, Atom atom, unsigned long *nelements,
 /*
  * Gets an string property's value from a window.
  */
-bool XAtom::getStringValue(Window win, Atom atom, unsigned long *nelements,
-                     std::string &value) const {
+bool XAtom::getStringValue(Window win, Atom atom, std::string &value) const {
   unsigned char *data;
-  bool ret = XAtom::getValue(win, atom, XA_STRING, nelements, &data, 8);
+  unsigned long nelements;
+  bool ret = XAtom::getValue(win, atom, XA_STRING, &nelements, &data, 8);
   if (ret)
     value = reinterpret_cast<char*>(data);
   return ret;