feature: set_timeout() now returns the result of the operation. thank…

…s Guanlan Dai for the report.

README.markdown

@@ -201,10 +201,12 @@ The `flags` parameter is optional, defaults to `0`.
 
 set_timeout
 ----------
-`syntax: memc:set_timeout(time)`
+`syntax: ok, err = memc:set_timeout(time)`
 
 Sets the timeout (in ms) protection for subsequent operations, including the `connect` method.
 
+Returns 1 when successful and nil plus a string describing the error otherwise.
+
 [Back to TOC](#table-of-contents)
 
 set_keepalive

lib/resty/memcached.lua

@@ -56,7 +56,8 @@ function _M.set_timeout(self, timeout)
         return nil, "not initialized"
     end
 
-    return sock:settimeout(timeout)
+    sock:settimeout(timeout)
+    return 1
 end
 
 

t/sanity.t

@@ -30,7 +30,7 @@ __DATA__
             local memcached = require "resty.memcached"
             local memc = memcached:new()
 
-            memc:set_timeout(1000) -- 1 sec
+            assert(memc:set_timeout(1000)) -- 1 sec
 
             local ok, err = memc:connect("127.0.0.1", $TEST_NGINX_MEMCACHED_PORT)
             if not ok then