Document memcached.sess_binary_protocol

And remove a TODO found in the code for allowing users to configure
whether or not to use binary protocols for sessions.

For issue #233

Author: TysonAndre

memcached.ini

@@ -37,7 +37,7 @@ memcached.sess_prefix = "memc.sess.key."
 ; default is Off
 memcached.sess_consistent_hash = Off
 
-; Allow failed memcached server to automatically be removed 
+; Allow failed memcached server to automatically be removed
 memcached.sess_remove_failed = 1
 
 ; Write data to a number of additional memcached servers
@@ -49,15 +49,17 @@ memcached.sess_remove_failed = 1
 ; which could have old data or no data at all
 memcached.sess_number_of_replicas = 0
 
-; memcached session binary mode
-; libmemcached replicas only work if binary mode is enabled
-memcached.sess_binary = Off
+; Use the memcached binary protocol for memcached sessions (Instead of the text protocol)
+; libmemcached replicas work only if binary mode is enabled.
+; However, certain proxies (such as twemproxy) will work only if the binary protocol is disabled.
+; Default is On. In older versions of php-memcached, this setting was Off and was called memcached.sess_binary.
+memcached.sess_binary_protocol = On
 
 ; memcached session replica read randomize
 memcached.sess_randomize_replica_read = Off
 
 ; memcached connect timeout value
-; In non-blocking mode this changes the value of the timeout 
+; In non-blocking mode this changes the value of the timeout
 ; during socket connection in milliseconds. Specifying -1 means an infinite timeout.
 memcached.sess_connect_timeout = 1000
 

php_memcached.c

@@ -19,7 +19,6 @@
 /* TODO
  * - set LIBKETAMA_COMPATIBLE as the default?
  * - fix unserialize(serialize($memc))
- * - ability to set binary protocol for sessions
  */
 
 #include "php_memcached.h"