Feature/napi

.travis.yml

@@ -1,38 +1,25 @@
-sudo: false
+os:
+  - linux
+  - windows
+  - osx
 
-env:
-  - CXX=g++-4.8
-addons:
-  apt:
-    sources:
-      - ubuntu-toolchain-r-test
-    packages:
-      - g++-4.8
+sudo: false
 
 language: node_js
 
 node_js:
-  - "0.8"
-  - "0.10"
-  - "0.12"
-  - "1"
-  - "2"
-  - "3"
-  - "4"
-  - "5"
-  - "6"
-  - "7"
+  - "8"
+  - "10"
+  - "node"
 
 install:
-  - PATH="`npm bin`:`npm bin -g`:$PATH"
-  # Node 0.8 comes with a too obsolete npm
-  - if [[ "`node --version`" =~ ^v0\.8\. ]]; then npm install -g [email protected] ; fi
-  # Install dependencies and build
   - npm install
 
+# note, the original master branch using nan did not have a travis
+# job on windows and when unit tested was found to have breaking
+# tests. If these were fixed existing work arounds would be broken
+# on windows.
 script:
-  # Output useful info for debugging
   - node --version
   - npm --version
-  # Run tests
-  - npm test
+  - if [[ "$TRAVIS_OS_NAME" != "windows" ]]; then npm test ; fi

appveyor.yml

@@ -7,15 +7,10 @@ environment:
   # Test against these versions of Node.js and io.js
   matrix:
     # node.js
-    - nodejs_version: "0.10"
-    - nodejs_version: "0.12"
-    # io.js
-    - nodejs_version: "2"
-    - nodejs_version: "3.2"
-    - nodejs_version: "4"
-    - nodejs_version: "5"
-    - nodejs_version: "6"
-    - nodejs_version: "7"
+    - nodejs_version: "8"
+    - nodejs_version: "9"
+    - nodejs_version: "10"
+    - nodejs_version: "11"
 
 platform:
   - x86

binding.gyp

@@ -1,30 +1,37 @@
 {
-  'targets': [
-    {
-      'target_name': 'time',
-      'include_dirs': [
-        '<!(node -e "require(\'nan\')")'
-      ],
-      'sources': [ 'src/time.cc' ],
-      'conditions': [
-        ['OS=="mac"', {
-          'defines': [
-            '__DARWIN_UNIX03', # For char* timezone
-            'HAVE_TM_GMTOFF'
-          ]
-        }],
-        ['OS=="linux"', {
-          'defines': [
-            'HAVE_TM_GMTOFF'
-          ]
-        }],
-        ['OS=="solaris"', {
-          'defines': [
-            'HAVE_ALTZONE',
-            'HAVE_TIMEZONE'
-          ]
-        }]
-      ]
-    }
-  ]
+    'targets': [{
+        'target_name': 'time',
+        'sources': [ 'src/time_napi.cc' ],
+        'include_dirs': ["<!@(node -p \"require('node-addon-api').include\")"],
+        'dependencies': ["<!(node -p \"require('node-addon-api').gyp\")"],
+        'cflags!': [ '-fno-exceptions' ],
+        'cflags_cc!': [ '-fno-exceptions' ],
+        'xcode_settings': {
+        'GCC_ENABLE_CPP_EXCEPTIONS': 'YES',
+        'CLANG_CXX_LIBRARY': 'libc++',
+        'MACOSX_DEPLOYMENT_TARGET': '10.7'
+        },
+        'msvs_settings': {
+        'VCCLCompilerTool': { 'ExceptionHandling': 1 },
+        },
+        'conditions': [
+                   ['OS=="mac"', {
+                     'defines': [
+                       '__DARWIN_UNIX03', # For char* timezone
+                       'HAVE_TM_GMTOFF'
+                     ]
+                   }],
+                   ['OS=="linux"', {
+                     'defines': [
+                       'HAVE_TM_GMTOFF'
+                     ]
+                   }],
+                   ['OS=="solaris"', {
+                     'defines': [
+                       'HAVE_ALTZONE',
+                       'HAVE_TIMEZONE'
+                     ]
+                   }]
+                 ]
+     }]
 }

index.js

@@ -5,12 +5,13 @@
 var debug = require('debug')('time')
   , fs = require('fs')
   , path = require('path')
-  , bindings = require('bindings')('time.node')
+  , time =  require('bindings')('time')
   , MILLIS_PER_SECOND = 1000
   , DAYS_OF_WEEK = ['Sunday','Monday','Tuesday','Wednesday','Thursday','Friday','Saturday']
   , MONTHS = ['January','February','March','April','May','June','July','August','September','October','November','December']
   , TZ_BLACKLIST = [ 'SystemV', 'Etc' ];
 
+var bindings = new time.Time();
 /**
  * Extends a "Date" constructor with node-time's extensions.
  * By default, `time.Date` is extended with this function.
@@ -38,9 +39,14 @@ exports.currentTimezone = process.env.TZ;
  * Export the raw functions from the bindings.
  */
 
-exports.time = bindings.time;
-exports.localtime = bindings.localtime;
-exports.mktime = bindings.mktime;
+exports.localtime = function(t) {
+  debug('calling native localtime');
+  return bindings.localtime(t);
+}
+exports.mktime = function(o) {
+  debug('calling native mktime');
+  return bindings.mktime(o);
+}
 
 /**
  * A "hack" of sorts to force getting our own Date instance.