From 6999fb3d1e342cc5bcd41c5ff41ff0c88ddfb250 Mon Sep 17 00:00:00 2001 From: Ben Noordhuis Date: Mon, 23 Jan 2012 21:36:48 +0100 Subject: [PATCH] dgram: make addMembership() and dropMembership() conform to v0.4 API - throw on error, don't return an error code --- lib/dgram.js | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/lib/dgram.js b/lib/dgram.js index 2e8071ca121..ee239558199 100644 --- a/lib/dgram.js +++ b/lib/dgram.js @@ -260,7 +260,9 @@ Socket.prototype.addMembership = function(multicastAddress, throw new Error('multicast address must be specified'); } - return this._handle.addMembership(multicastAddress, interfaceAddress); + if (this._handle.addMembership(multicastAddress, interfaceAddress)) { + throw new errnoException(errno, 'addMembership'); + } }; @@ -272,7 +274,9 @@ Socket.prototype.dropMembership = function(multicastAddress, throw new Error('multicast address must be specified'); } - return this._handle.dropMembership(multicastAddress, interfaceAddress); + if (this._handle.dropMembership(multicastAddress, interfaceAddress)) { + throw new errnoException(errno, 'dropMembership'); + } };