Used std::string instead of char and sprintf for constructing an error

message.

diff --git a/libmemcached/memcached.hpp b/libmemcached/memcached.hpp
index bb8a9732d9b8614614f2b501650bf67a570018df..c42eba03c89f2be454133769c437b4b0b85081ab 100644 (file)
--- a/libmemcached/memcached.hpp
+++ b/libmemcached/memcached.hpp
@@ -446,9 +446,8 @@ public:
       retval= set(it->first, it->second, expiration, flags);
       if (retval == false)
       {
-        char err_buff[64];
-        sprintf(err_buff,  "There was an error setting the key %s",
-          it->first.c_str());
+        std::string err_buff("There was an error setting the key ");
+        err_buff.append(it->first);
         throw(Error(err_buff, false));
       }
       ++it;

diff --git a/tests/cpp_example.cc b/tests/cpp_example.cc
index 5454f0203512651c7fe392719b9351dc22e93cc9..b96aaf4e1c23bf7a838f5f7904dc619e3dbe65f2 100644 (file)
--- a/tests/cpp_example.cc
+++ b/tests/cpp_example.cc
@@ -167,6 +167,7 @@ void setProduct(const string &key, const Product &product)
 
 int main()
 {
+#if 0
   Product pad(1, 5.0);
   const string key("padraig");
   cout << "Going to set an object in the cache..." << endl;
@@ -190,5 +191,12 @@ int main()
     cout << "product " << (*iter).getId() << " costs " << (*iter).getPrice() << endl;
     ++iter;
   }
+#endif
+  Memcache first_client("127.0.0.1:11211");
+  Memcache second_client("127.0.0.1", 11211);
+  //first_client.set("key", some_vector_of_chars, expiry, flags);
+  //first_client.get("key", vector_to_fill_with_data);
+  //first_client.remove("key");
+  first_client.addServer("192.168.1.1", 11211);
   return 0;
 }