summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorPierre Ossman <ossman@cendio.se>2018-09-06 16:53:40 +0200
committerPierre Ossman <ossman@cendio.se>2018-09-06 16:53:40 +0200
commite777765320f594c6e3ca44929cdbc2e41949ecc1 (patch)
tree4fe1264c87c0e3143f11c38cb7a6cbee105714ae
parentd80d9d3731725f7b0585bc1450385ed323b37492 (diff)
downloadnovnc-e777765320f594c6e3ca44929cdbc2e41949ecc1.tar.gz
Enforce function names
-rw-r--r--.eslintrc1
-rw-r--r--app/error-handler.js6
-rw-r--r--tests/.eslintrc4
-rwxr-xr-xutils/use_require.js2
-rw-r--r--utils/use_require_helpers.js2
5 files changed, 9 insertions, 6 deletions
diff --git a/.eslintrc b/.eslintrc
index f0c27a9..9f7d67f 100644
--- a/.eslintrc
+++ b/.eslintrc
@@ -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) => {