Merge pull request #1007 from juanjoDiaz/remove_jshints_comments

Remove jshint comments

app/ui.js

@@ -8,9 +8,6 @@
  * See README.md for usage and integration instructions.
  */
 
-/* jslint white: false, browser: true */
-/* global window, document.getElementById, Util, WebUtil, RFB, Display */
-
 import * as Log from '../core/util/logging.js';
 import _, { l10n } from './localization.js';
 import { isTouchDevice } from '../core/util/browsers.js';

app/webutil.js

@@ -7,9 +7,6 @@
  * See README.md for usage and integration instructions.
  */
 
-/*jslint bitwise: false, white: false, browser: true, devel: true */
-/*global Util, window, document */
-
 import { init_logging as main_init_logging } from '../core/util/logging.js';
 
 // init log level reading the logging HTTP param

core/base64.js

@@ -4,8 +4,6 @@
 
 // From: http://hg.mozilla.org/mozilla-central/raw-file/ec10630b1a54/js/src/devtools/jint/sunspider/string-base64.js
 
-/*jslint white: false */
-/*global console */
 import * as Log from './util/logging.js';
 
 export default {
@@ -48,7 +46,6 @@ export default {
     },
 
     /* Convert Base64 data to a string */
-    /* jshint -W013 */
     toBinaryTable : [
         -1,-1,-1,-1, -1,-1,-1,-1, -1,-1,-1,-1, -1,-1,-1,-1,
         -1,-1,-1,-1, -1,-1,-1,-1, -1,-1,-1,-1, -1,-1,-1,-1,
@@ -59,7 +56,6 @@ export default {
         -1,26,27,28, 29,30,31,32, 33,34,35,36, 37,38,39,40,
         41,42,43,44, 45,46,47,48, 49,50,51,-1, -1,-1,-1,-1
     ],
-    /* jshint +W013 */
 
     decode: function (data, offset) {
         "use strict";

core/des.js

@@ -75,13 +75,10 @@
  * fine Java utilities: http://www.acme.com/java/
  */
 
-/* jslint white: false */
-
 export default function DES(passwd) {
     "use strict";
 
     // Tables, permutations, S-boxes, etc.
-    // jshint -W013
     var PC2 = [13,16,10,23, 0, 4, 2,27,14, 5,20, 9,22,18,11, 3,
                25, 7,15, 6,26,19,12, 1,40,51,30,36,46,54,29,39,
                50,44,32,47,43,48,38,55,33,52,45,41,49,35,28,31 ],
@@ -89,7 +86,6 @@ export default function DES(passwd) {
         z = 0x0, a,b,c,d,e,f, SP1,SP2,SP3,SP4,SP5,SP6,SP7,SP8,
         keys = [];
 
-    // jshint -W015
     a=1<<16; b=1<<24; c=a|b; d=1<<2; e=1<<10; f=d|e;
     SP1 = [c|e,z|z,a|z,c|f,c|d,a|f,z|d,a|z,z|e,c|e,c|f,z|e,b|f,c|d,b|z,z|d,
            z|f,b|e,b|e,a|e,a|e,c|z,c|z,b|f,a|d,b|d,b|d,a|d,z|z,z|f,a|f,b|z,
@@ -130,7 +126,6 @@ export default function DES(passwd) {
            c|z,b|d,b|e,z|f,a|e,a|d,c|d,c|e,z|f,z|z,z|z,c|d,b|d,b|e,a|f,a|z,
            a|f,a|z,c|e,z|e,z|d,c|d,z|e,a|f,b|e,z|d,b|d,c|z,c|d,b|z,a|z,b|f,
            z|z,c|f,a|d,b|d,c|z,b|e,b|f,z|z,c|f,a|e,a|e,z|f,z|f,a|d,b|z,c|e];
-    // jshint +W013,+W015
 
     // Set the key.
     function setKeys(keyBlock) {

core/display.js

@@ -7,9 +7,6 @@
  * See README.md for usage and integration instructions.
  */
 
-/*jslint browser: true, white: false */
-/*global Util, Base64, changeCursor */
-
 import * as Log from './util/logging.js';
 import Base64 from "./base64.js";
 

core/input/keyboard.js

@@ -5,9 +5,6 @@
  * Licensed under MPL 2.0 or any later version (see LICENSE.txt)
  */
 
-/*jslint browser: true, white: false */
-/*global window, Util */
-
 import * as Log from '../util/logging.js';
 import { stopEvent } from '../util/events.js';
 import * as KeyboardUtil from "./util.js";

core/input/mouse.js

@@ -5,9 +5,6 @@
  * Licensed under MPL 2.0 or any later version (see LICENSE.txt)
  */
 
-/*jslint browser: true, white: false */
-/*global window, Util */
-
 import * as Log from '../util/logging.js';
 import { isTouchDevice } from '../util/browsers.js';
 import { setCapture, stopEvent, getPointerEvent } from '../util/events.js';

core/util/logging.js

@@ -26,7 +26,6 @@ export function init_logging (level) {
 
     Debug = Info = Warn = Error = function (msg) {};
     if (typeof window.console !== "undefined") {
-        /* jshint -W086 */
         switch (level) {
             case 'debug':
                 Debug = console.debug.bind(window.console);
@@ -41,7 +40,6 @@ export function init_logging (level) {
             default:
                 throw new Error("invalid logging type '" + level + "'");
         }
-        /* jshint +W086 */
     }
 };
 export function get_logging () {

core/websock.js

@@ -14,9 +14,6 @@
 
 import * as Log from './util/logging.js';
 
-/*jslint browser: true, bitwise: true */
-/*global Util*/
-
 export default function Websock() {
     "use strict";
 

tests/test.display.js

@@ -1,4 +1,3 @@
-/* jshint expr: true */
 var expect = chai.expect;
 
 import Base64 from '../core/base64.js';

tests/test.keyboard.js

@@ -12,7 +12,6 @@ function isEdge() {
     return navigator && !!(/edge/i).exec(navigator.userAgent);
 }
 
-/* jshint newcap: false, expr: true */
 describe('Key Event Handling', function() {
     "use strict";
 

tests/test.localization.js

@@ -1,5 +1,3 @@
-/* jshint expr: true */
-
 var assert = chai.assert;
 var expect = chai.expect;
 

tests/test.mouse.js

@@ -6,7 +6,6 @@ import sinon from '../vendor/sinon.js';
 import Mouse from '../core/input/mouse.js';
 import * as eventUtils from '../core/util/events.js';
 
-/* jshint newcap: false, expr: true */
 describe('Mouse Event Handling', function() {
     "use strict";
 

tests/test.rfb.js

@@ -1,4 +1,3 @@
-/* jshint expr: true */
 var assert = chai.assert;
 var expect = chai.expect;
 

tests/test.util.js

@@ -1,5 +1,3 @@
-/* jshint expr: true */
-
 var assert = chai.assert;
 var expect = chai.expect;
 

tests/test.websock.js

@@ -1,4 +1,3 @@
-/* jshint expr: true */
 var assert = chai.assert;
 var expect = chai.expect;
 

vendor/pako/lib/zlib/deflate.js

@@ -280,7 +280,7 @@ function longest_match(s, cur_match) {
      * the 256th check will be made at strstart+258.
      */
     do {
-      /*jshint noempty:false*/
+      // Do nothing
     } while (_win[++scan] === _win[++match] && _win[++scan] === _win[++match] &&
              _win[++scan] === _win[++match] && _win[++scan] === _win[++match] &&
              _win[++scan] === _win[++match] && _win[++scan] === _win[++match] &&
@@ -892,7 +892,7 @@ function deflate_rle(s, flush) {
       if (prev === _win[++scan] && prev === _win[++scan] && prev === _win[++scan]) {
         strend = s.strstart + MAX_MATCH;
         do {
-          /*jshint noempty:false*/
+          // Do nothing
         } while (prev === _win[++scan] && prev === _win[++scan] &&
                  prev === _win[++scan] && prev === _win[++scan] &&
                  prev === _win[++scan] && prev === _win[++scan] &&