diff options
author | Pierre Ossman <ossman@cendio.se> | 2018-09-06 16:53:40 +0200 |
---|---|---|
committer | Pierre Ossman <ossman@cendio.se> | 2018-09-06 16:53:40 +0200 |
commit | e777765320f594c6e3ca44929cdbc2e41949ecc1 (patch) | |
tree | 4fe1264c87c0e3143f11c38cb7a6cbee105714ae | |
parent | d80d9d3731725f7b0585bc1450385ed323b37492 (diff) | |
download | novnc-e777765320f594c6e3ca44929cdbc2e41949ecc1.tar.gz |
Enforce function names
-rw-r--r-- | .eslintrc | 1 | ||||
-rw-r--r-- | app/error-handler.js | 6 | ||||
-rw-r--r-- | tests/.eslintrc | 4 | ||||
-rwxr-xr-x | utils/use_require.js | 2 | ||||
-rw-r--r-- | utils/use_require_helpers.js | 2 |
5 files changed, 9 insertions, 6 deletions
@@ -32,5 +32,6 @@ "comma-spacing": ["error"], "comma-style": ["error"], "func-call-spacing": ["error"], + "func-names": ["error"], } } diff --git a/app/error-handler.js b/app/error-handler.js index 96dc926..f02f535 100644 --- a/app/error-handler.js +++ b/app/error-handler.js @@ -5,7 +5,7 @@ // No ES6 can be used in this file since it's used for the translation /* eslint-disable prefer-arrow-callback */ -(function() { +(function _scope() { "use strict"; // Fallback for all uncought errors @@ -53,6 +53,6 @@ // from being printed to the browser console. return false; } - window.addEventListener('error', function (evt) { handleError(evt, evt.error); }); - window.addEventListener('unhandledrejection', function (evt) { handleError(evt.reason, evt.reason); }); + window.addEventListener('error', function onerror(evt) { handleError(evt, evt.error); }); + window.addEventListener('unhandledrejection', function onreject(evt) { handleError(evt.reason, evt.reason); }); })(); diff --git a/tests/.eslintrc b/tests/.eslintrc index fa6fd40..545fa2e 100644 --- a/tests/.eslintrc +++ b/tests/.eslintrc @@ -8,6 +8,8 @@ "sinon": false }, "rules": { - "prefer-arrow-callback": 0 + "prefer-arrow-callback": 0, + // Too many anonymous callbacks + "func-names": "off", } } diff --git a/utils/use_require.js b/utils/use_require.js index f301e1d..9d6f416 100755 --- a/utils/use_require.js +++ b/utils/use_require.js @@ -43,7 +43,7 @@ no_copy_files.forEach(file => no_transform_files.add(file)); // util.promisify requires Node.js 8.x, so we have our own function promisify(original) { - return function () { + return function promise_wrap() { const args = Array.prototype.slice.call(arguments); return new Promise((resolve, reject) => { original.apply(this, args.concat((err, value) => { diff --git a/utils/use_require_helpers.js b/utils/use_require_helpers.js index 4daa81f..90342cb 100644 --- a/utils/use_require_helpers.js +++ b/utils/use_require_helpers.js @@ -4,7 +4,7 @@ const path = require('path'); // util.promisify requires Node.js 8.x, so we have our own function promisify(original) { - return function () { + return function promise_wrap() { const args = Array.prototype.slice.call(arguments); return new Promise((resolve, reject) => { original.apply(this, args.concat((err, value) => { |