more windows work

Author: joeferner

mnm.js

@@ -15,8 +15,12 @@ ociLibDir = process.env["OCI_LIB_DIR"] || "/usr/lib/oracle/11.2/client/lib/";
 builder.failIfNotExists(ociLibDir, 'Could not find "%s" check OCI_LIB_DIR environment variable.');
 builder.appendLinkerSearchDir(ociLibDir);
 
-builder.appendLinkerLibrary("occi");
-builder.appendLinkerLibrary("clntsh");
+if(process.platform == 'win32') {
+  builder.appendLinkerLibrary("oci");
+} else {
+  builder.appendLinkerLibrary("occi");
+  builder.appendLinkerLibrary("clntsh");
+}
 if (path.existsSync(path.join(ociLibDir, "libnnz10.dylib"))
     || path.existsSync(path.join(ociLibDir, "libnnz10.so"))) {
   builder.appendLinkerLibrary("nnz10");

src/executeBaton.cpp

@@ -40,7 +40,7 @@ ExecuteBaton::~ExecuteBaton() {
 }
 
 double CallDateMethod(v8::Local<v8::Date> date, const char* methodName) {
-  Handle<Value> args[0];
+  Handle<Value> args[1]; // should be zero but on windows the compiler will not allow a zero length array
   Local<Value> result = Local<Function>::Cast(date->Get(String::New(methodName)))->Call(date, 0, args);
   return Local<Number>::Cast(result)->Value();
 }