merge node-usb-detection and serial-port into one native bindings. (#297)

* merge node-usb-detection and serial-port into one native bindings.

* fix eslint.

* cannot use arrow function in prototype, see https://rainsoft.io/when-not-to-use-arrow-functions-in-javascript/

* disable usb detector in linux since it is not supported now.

* add all native dlls

* disable loading error when loading dll.

* test travis-ci error for dll loading

* use arch in glob search.

* add editorconfig back

Author: andxu

.editorconfig

@@ -1,14 +1,14 @@
-# EditorConfig is awesome: http://EditorConfig.org
-
-# top-most EditorConfig file
-root = true
-
-# Tab indentation
-[*]
-indent_style = space
-indent_size = 4
-
-# The indent size used in the `package.json` file cannot be changed
-# https://github.com/npm/npm/pull/3180#issuecomment-16336516
-[{.travis.yml,npm-shrinkwrap.json,package.json}]
-indent_size = 2
+# EditorConfig is awesome: http://EditorConfig.org
+
+# top-most EditorConfig file
+root = true
+
+# Tab indentation
+[*]
+indent_style = space
+indent_size = 2
+
+# The indent size used in the `package.json` file cannot be changed
+# https://github.com/npm/npm/pull/3180#issuecomment-16336516
+[{.travis.yml,npm-shrinkwrap.json,package.json}]
+indent_size = 2

gulpfile.js

@@ -1,4 +1,5 @@
 const gulp = require("gulp");
+const eslint = require('gulp-eslint');
 const tslint = require("gulp-tslint");
 const gutil = require("gulp-util");
 const ts = require("gulp-typescript");
@@ -18,6 +19,13 @@ gulp.task("tslint", () => {
         .pipe(tslint.report());
 });
 
+gulp.task("eslint", () => {
+    return gulp.src(["./vendor/**/*.js","!**/node_modules/**"])
+        .pipe(eslint())
+        .pipe(eslint.format())
+        .pipe(eslint.failAfterError());
+});
+
 gulp.task("html-webpack", (done) => {
     const config = require("./html/webpack.config.js");
     config.context = `${__dirname}/html`;

package.json

@@ -178,7 +178,15 @@
     "@types/node": "^6.0.40",
     "@types/winreg": "^1.2.30",
     "del": "^2.2.2",
+    "eslint": "^3.19.0",
+    "eslint-config-standard": "^10.2.1",
+    "eslint-plugin-import": "^2.2.0",
+    "eslint-plugin-node": "^4.2.2",
+    "eslint-plugin-promise": "^3.5.0",
+    "eslint-plugin-require-path-exists": "^1.1.7",
+    "eslint-plugin-standard": "^3.0.1",
     "gulp": "^3.9.1",
+    "gulp-eslint": "^3.0.1",
     "gulp-sourcemaps": "^2.4.1",
     "gulp-tslint": "^7.0.1",
     "gulp-typescript": "^3.1.5",

src/serialmonitor/serialportctrl.ts

@@ -26,7 +26,7 @@ export class SerialPortCtrl {
         });
     }
 
-    private static serialport = require("../../../vendor/serialport-native");
+    private static serialport = require("../../../vendor/node-usb-native").SerialPort;
 
     private _currentPort: string;
     private _currentBaudRate: number;

src/serialmonitor/usbDetector.ts

@@ -30,7 +30,7 @@ export class UsbDetector {
         if (os.platform() === "linux") {
             return;
         }
-        this._usbDector = require("../../../vendor/node-usb-detection");
+        this._usbDector = require("../../../vendor/node-usb-native").detector;
 
         if (!this._usbDector) {
             return;

test/extension.test.ts

@@ -60,7 +60,7 @@ suite("Arduino Extension Tests", () => {
         // When running test on osx, the vscode instance is hanging there after tests finished and cause mocha timeout.
         // As a workaround, closing usb-detection process manually would make test window exit normally.
         if (os.platform() !== "linux") {
-            const usbDector = require("../../vendor/node-usb-detection");
+            const usbDector = require("../../vendor/node-usb-native").detector;
             usbDector.stopMonitoring();
         }
     });