Updated the fetch method in the C++ interface and added doxygen comments to

it.

diff --git a/libmemcached/memcached.hpp b/libmemcached/memcached.hpp
index ecb474aa0fc117372e4cd1a9537a9dc1c1346742..55941f10bad9f7fe5cb94c105032241a95045fca 100644 (file)
--- a/libmemcached/memcached.hpp
+++ b/libmemcached/memcached.hpp
@@ -91,16 +91,23 @@ public:
     return memcached_strerror(NULL, rc);
   }
 
+  /**
+   * Fetches an individual value from the server.
+   *
+   * @param[in] key key of object to fetch
+   * @param[out] ret_val store returned object in this vector
+   * @return true on success; false on failure
+   */
   bool fetch(std::string &key, 
-             std::vector<char> &ret_val,
-             uint32_t *flags,
-             memcached_return *rc)
+             std::vector<char> &ret_val)
   {
     char ret_key[MEMCACHED_MAX_KEY];
     size_t value_length= 0;
     size_t key_length= 0;
+    memcached_return rc;
+    uint32_t flags= 0;
     char *value= memcached_fetch(&memc, ret_key, &key_length,
-                                 &value_length, flags, rc);
+                                 &value_length, &flags, &rc);
     if (value && ret_val.empty())
     {
       ret_val.reserve(value_length);

diff --git a/tests/plus.cpp b/tests/plus.cpp
index 3431fd188dd0b2a6dc5ecf3857f35eb8700756f7..78e4670f06b06da2b2efe6e977a2b933bae7b7ea 100644 (file)
--- a/tests/plus.cpp
+++ b/tests/plus.cpp
@@ -212,7 +212,6 @@ test_return mget_test(memcached_st *memc)
   values.push_back(&val1);
   values.push_back(&val2);
   values.push_back(&val3);
-  uint32_t flags;
 
   string return_key;
   vector<char> return_value;
@@ -224,8 +223,7 @@ test_return mget_test(memcached_st *memc)
   rc= mc.mget(keys);
   assert(rc == true);
 
-  while (mc.fetch(return_key, return_value, 
-                  &flags, &mc_rc))
+  while (mc.fetch(return_key, return_value))
   {
     assert(return_value.size() != 0);
     return_value.clear();
@@ -238,10 +236,8 @@ test_return mget_test(memcached_st *memc)
   rc= mc.mget(keys);
   assert(rc == true);
 
-  while ((mc.fetch(return_key, return_value, 
-                   &flags, &mc_rc)))
+  while ((mc.fetch(return_key, return_value)))
   {
-    assert(mc_rc == MEMCACHED_SUCCESS);
     assert(return_key.length() == return_value.size());
     assert(!memcmp(&return_value[0], return_key.c_str(), return_value.size()));
   }