Add support for multi-reply handlers

sidorares · Apr 27, 2015 · 624e8df · 624e8df
1 parent e7d9ba7
commit 624e8df
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/lib/xcore.js b/lib/xcore.js
@@ -506,9 +506,11 @@ XClient.prototype.expectReplyHeader = function()
 
                     var result = unpack.call(client, data, opt_data);
                     var callback = handler[1];
-                    callback(null, result);
-                    // TODO: add multiple replies flag and delete handler only after last reply (eg ListFontsWithInfo)
-                    delete client.replies[seq_num];
+                    var multiReply = handler[2];
+                    var handled = callback(null, result);
+                    if (!multiReply || handled) {
+                        delete client.replies[seq_num];
+                    }
                 }
                 // wait for new packet from server
                 client.expectReplyHeader();