fix for buffered_writes -- now drain event works

index.js

@@ -286,7 +286,7 @@ RedisClient.prototype.send_offline_queue = function () {
     this.offline_queue = new Queue();
     // Even though items were shifted off, Queue backing store still uses memory until next add, so just get a new Queue
     
-    if (buffered_writes === 0) {
+    if (!buffered_writes) {
         this.should_buffer = false;
         this.emit("drain");
     }

tests/stress/pubsub/pub.js

@@ -1,13 +1,33 @@
 'use strict';
 
+var freemem = require('os').freemem;
 var codec = require('../codec');
 
-var pub = require('redis').createClient()
+var sent = 0;
+
+var pub = require('redis').createClient(null, null, {
+	command_queue_high_water: 5,
+	command_queue_low_water: 1
+})
 .on('ready', function() {
-		while (true) {
-			pub.publish('timeline', codec.encode({
-				cmd: Math.random(),
-				data: Math.random()
-			}));
-		}
+	this.emit('drain');
+})
+.on('drain', function() {
+	process.nextTick(exec);
 });
+
+var payload = '1'; for (var i = 0; i < 10; ++i) payload += payload;
+
+function exec() {
+	pub.publish('timeline', codec.encode({ foo: payload }));
+	++sent;
+	if (!pub.should_buffer) {
+		process.nextTick(exec);
+	}
+}
+
+exec();
+
+setInterval(function() {
+	console.log('sent', sent, 'free', freemem(), 'cmdqlen', pub.command_queue.length, 'offqlen', pub.offline_queue.length);
+}, 1000);