Re-enable jshint with a minimal configuration

.jshintignore

@@ -9,27 +9,3 @@ lib-jitsi-meet.min.js
 # The following are checked by ESLint and take advantage of advanced language
 # features such as Facebook Flow which are not supported by JSHint.
 flow-typed/
-
-JitsiConference.js
-JitsiConnection.js
-JitsiMeetJS.js
-JitsiParticipant.js
-modules/RTC/JitsiLocalTrack.js
-modules/RTC/JitsiRemoteTrack.js
-modules/RTC/JitsiTrack.js
-modules/RTC/DataChannels.js
-modules/RTC/adapter.screenshare.js
-modules/RTC/RTC.js
-modules/RTC/RTCUtils.js
-modules/RTC/TraceablePeerConnection.js
-modules/statistics/LocalStatsCollector.js
-modules/statistics/RTPStatsCollector.js
-modules/statistics/statistics.js
-modules/xmpp/ChatRoom.js
-modules/xmpp/JingleSessionPC.js
-modules/xmpp/LocalSSRCReplacement.js
-modules/xmpp/SDP.js
-modules/xmpp/SDPUtil.js
-modules/xmpp/strophe.emuc.js
-modules/xmpp/strophe.jingle.js
-modules/xmpp/xmpp.js

.jshintrc

@@ -1,20 +1,10 @@
 {
-    // Refer to http://jshint.com/docs/options/ for an exhaustive list of options
-    "asi": false, // true: Tolerate Automatic Semicolon Insertion (no semicolons)
     "browser": true, // Web Browser (window, document, etc)
-    "curly": false, // true: Require {} for every new block or scope
     "esversion": 6,
-    "evil": true, // true: Tolerate use of `eval` and `new Function()`
     "expr": true, // true: Tolerate `ExpressionStatement` as Programs
-    "indent": 4, // {int} Number of spaces to use for indentation
+    "jasmine": true,
     "latedef": false, //This option prohibits the use of a variable before it was defined
     "laxbreak": true, //Ignore line breaks around "=", "==", "&&", etc.
     "loopfunc": true, // true: Tolerate functions being defined in loops
-    "maxlen": 80, // {int} Max number of characters per line
-    "newcap": true, // true: Require capitalization of all constructor functions e.g. `new F()`
-    "node": true, // Node.js
-    "trailing": true,
-    "undef": true, // true: Require all non-global variables to be declared (prevents global leaks)
-    "white": true,
-    "jasmine": true
+    "node": true // Node.js
 }

JitsiConferenceEventManager.js

@@ -500,8 +500,9 @@ JitsiConferenceEventManager.prototype.setupXMPPListeners = function() {
 
     conference.xmpp.caps.addListener(XMPPEvents.PARTCIPANT_FEATURES_CHANGED,
         from => {
-            const participant = conference.getParticipantId(
-                Strophe.getResourceFromJid(from));
+            const participant
+                = conference.getParticipantId(
+                    Strophe.getResourceFromJid(from));
 
             if (participant) {
                 conference.eventEmitter.emit(
@@ -510,9 +511,11 @@ JitsiConferenceEventManager.prototype.setupXMPPListeners = function() {
             }
         });
     conference.xmpp.addListener(
-        XMPPEvents.CALL_INCOMING, conference.onIncomingCall.bind(conference));
+        XMPPEvents.CALL_INCOMING,
+        conference.onIncomingCall.bind(conference));
     conference.xmpp.addListener(
-        XMPPEvents.CALL_ENDED, conference.onCallEnded.bind(conference));
+        XMPPEvents.CALL_ENDED,
+        conference.onCallEnded.bind(conference));
 
     conference.xmpp.addListener(XMPPEvents.START_MUTED_FROM_FOCUS,
         (audioMuted, videoMuted) => {

modules/RTC/JitsiRemoteTrack.js

@@ -1,5 +1,3 @@
-/* global */
-
 const JitsiTrack = require('./JitsiTrack');
 
 import * as JitsiTrackEvents from '../../JitsiTrackEvents';

modules/RTC/RTCUtils.js

@@ -2,9 +2,9 @@
           __filename,
           attachMediaStream,
           MediaStreamTrack,
-          RTCIceCandidate,
+          RTCIceCandidate: true,
           RTCPeerConnection,
-          RTCSessionDescription,
+          RTCSessionDescription: true,
           mozRTCIceCandidate,
           mozRTCPeerConnection,
           mozRTCSessionDescription,