please the compiler god, and return something meaningful on connect()

Author: PAStheLoD

package.json

@@ -1,6 +1,6 @@
 {
 	"name": "rados",
-	"version": "0.2.101",
+	"version": "0.2.102",
 	"description" : "Ceph rados client for node.js",
 	"author" : "Laurent Barbe <[email protected]>",
  	"repository" : {

rados.cc

@@ -157,9 +157,10 @@ NAN_METHOD(Rados::connect) {
 
   if (err == 0) {
     obj->state = STATE_CONNECTED;
+    info.GetReturnValue().Set(Nan::True());
   }
 
-  info.GetReturnValue().Set(Nan::New<Number>(-err));
+  info.GetReturnValue().Set(Nan::False());
 }
 
 
@@ -530,6 +531,8 @@ NAN_METHOD(Ioctx::getxattr) {
     int ret = rados_getxattr(obj->ioctx, *oid, *name, temp_buffer, 0);
     if (ret < 0) {
       info.GetReturnValue().Set(Nan::Null());
+      return;
+
     } else {
       size = ret;
     }
@@ -668,7 +671,8 @@ void Ioctx::wait_complete(uv_work_t *req) {
 
 
 void Ioctx::callback_complete(uv_work_t *req) {
-  Isolate* isolate = Isolate::GetCurrent();
+  Isolate* VARIABLE_IS_NOT_USED isolate = Isolate::GetCurrent();
+
   Nan::HandleScope scope;
   AsyncData *asyncdata = (AsyncData *)req->data;
 

rados.h

@@ -93,5 +93,10 @@ class Ioctx : public Nan::ObjectWrap {
   static Nan::Persistent<v8::FunctionTemplate> constructor;
 };
 
+#ifdef __GNUC__
+#define VARIABLE_IS_NOT_USED __attribute__ ((unused))
+#else
+#define VARIABLE_IS_NOT_USED
+#endif
 
 #endif