enable 'array-bracket-spacing' rule

brvier · Sep 28, 2019 · eb16793 · eb16793
1 parent bcf6c56
commit eb16793
Show file tree

Hide file tree

Showing 4 changed files with 9 additions and 9 deletions.
diff --git a/.eslintrc.js b/.eslintrc.js
@@ -101,7 +101,7 @@ module.exports = {
     'no-global-assign': 2,
     'prefer-const': 2,
     'dot-notation': 0,
-    'array-bracket-spacing': 0,
+    'array-bracket-spacing': 2,
     'quote-props': 2,
     'no-void': 0,
     'no-useless-catch': 2,

diff --git a/src/loader/m3u8-parser.ts b/src/loader/m3u8-parser.ts
@@ -171,7 +171,7 @@ export default class M3U8Parser {
         // avoid sliced strings    https://github.com/video-dev/hls.js/issues/939
         const title = (' ' + result[2]).slice(1);
         frag.title = title || null;
-        frag.tagList.push(title ? [ 'INF', duration, title ] : [ 'INF', duration ]);
+        frag.tagList.push(title ? ['INF', duration, title] : ['INF', duration]);
       } else if (result[3]) { // url
         if (Number.isFinite(frag.duration)) {
           const sn = currentSN++;
@@ -227,7 +227,7 @@ export default class M3U8Parser {
 
         switch (result[i]) {
         case '#':
-          frag.tagList.push(value2 ? [ value1, value2 ] : [ value1 ]);
+          frag.tagList.push(value2 ? [value1, value2] : [value1]);
           break;
         case 'PLAYLIST-TYPE':
           level.type = value1.toUpperCase();

diff --git a/src/types/level.ts b/src/types/level.ts
@@ -63,7 +63,7 @@ export class Level {
   public unknownCodecs: string[] | undefined;
 
   constructor (data: LevelParsed) {
-    this.url = [ data.url ];
+    this.url = [data.url];
     this.attrs = data.attrs;
     this.bitrate = data.bitrate;
     this.details = data.details;

diff --git a/tests/unit/controller/abr-controller.js b/tests/unit/controller/abr-controller.js
@@ -12,11 +12,11 @@ describe('AbrController', function () {
     const hls = new Hls({ maxStarvationDelay: 4 });
     hls.levelController._levels = [
       { bitrate: 105000, name: '144', details: { totalduration: 4, fragments: [{}] } },
-      { bitrate: 246440, name: '240', details: { totalduration: 10, fragments: [ {} ] } },
-      { bitrate: 460560, name: '380', details: { totalduration: 10, fragments: [ {} ] } },
-      { bitrate: 836280, name: '480', details: { totalduration: 10, fragments: [ {} ] } },
-      { bitrate: 2149280, name: '720', details: { totalduration: 10, fragments: [ {} ] } },
-      { bitrate: 6221600, name: '1080', details: { totalduration: 10, fragments: [ {} ] } }
+      { bitrate: 246440, name: '240', details: { totalduration: 10, fragments: [{}] } },
+      { bitrate: 460560, name: '380', details: { totalduration: 10, fragments: [{}] } },
+      { bitrate: 836280, name: '480', details: { totalduration: 10, fragments: [{}] } },
+      { bitrate: 2149280, name: '720', details: { totalduration: 10, fragments: [{}] } },
+      { bitrate: 6221600, name: '1080', details: { totalduration: 10, fragments: [{}] } }
     ];
     const abrController = new AbrController(hls);
     abrController.bwEstimator = new EwmaBandWidthEstimator(hls, 15, 4, 5e5);