Remove "failed to set key" warning from setMulti

This was introduced in 6837d89494,
pull php-memcached-dev#214. I suspect it
may have been a left-over from debugging something. The test was later
changed in 6837d89 to expect the warning in question, although other
similar tests don't encounter such warning currently.

It appears no other Memcached methods emit a PHP Warning when they encounter
a write read or failure. Instead, they typically turn their return value
into boolean false, and provide details via getResultMessage().

The introduction of this warning since php-memcached 3.0 has led to a number
of confused consumers (locally php-memcached-dev#260, php-memcached-dev#409, php-memcached-dev#450, and more reports
within downstream issue trackers).

Closes php-memcached-dev#409.

Author: Krinkle

php_memcached.c

@@ -1907,9 +1907,8 @@ static void php_memc_setMulti_impl(INTERNAL_FUNCTION_PARAMETERS, zend_bool by_ke
 			str_key = zend_string_init(tmp_key, tmp_len, 0);
 		}
 
-		if (!s_memc_write_zval (intern, MEMC_OP_SET, server_key, str_key, value, expiration)) {
-			php_error_docref(NULL, E_WARNING, "failed to set key %s", ZSTR_VAL(str_key));
-		}
+		/* If this failed to set a key, intern stores the error for the return value */
+		s_memc_write_zval (intern, MEMC_OP_SET, server_key, str_key, value, expiration)
 
 		if (!skey) {
 			zend_string_release (str_key);

tests/experimental/setmulti_badserialize.phpt

@@ -38,10 +38,10 @@ try {
 
 	echo error_get_last()["message"], "\n";
 	echo $e->getMessage(), "\n";
+	echo $m->getResultMessage(), "\n";
 }
 var_dump($m->getByKey('kef', 'foo'));
 
 --EXPECT--
-Memcached::setMultiByKey(): failed to set key foo
 1234
 int(10)