Merge pull request #1679 from webtorrent/then

Use promise then() instead of catch() with callbacks

src/renderer/lib/state.js

@@ -151,8 +151,10 @@ function setupStateSaved (cb) {
   })
 
   Promise.all(tasks)
-    .then(() => cb(null, saved))
+    .then(
-    .catch(err => cb(err))
+      () => cb(null, saved),
+      err => cb(err)
+    )
 
   function createTorrentObject (t) {
     // TODO: Doing several fs.readFileSync calls during first startup is not ideal

src/renderer/webtorrent.js

@@ -358,11 +358,15 @@ function getAudioMetadata (infoHash, index) {
     : mm.parseStream(file.createReadStream(), file.name, options)
 
   onMetaData
-    .then(() => {
+    .then(
-      console.log(`metadata for file='${file.name}' completed.`)
+      () => console.log(`metadata for file='${file.name}' completed.`),
-    }).catch(function (err) {
+      err => {
-      return console.log('error getting audio metadata for ' + infoHash + ':' + index, err)
+        console.log(
-    })
+          `error getting audio metadata for ${infoHash}:${index}`,
+          err
+        )
+      }
+    )
 }
 
 function selectFiles (torrentOrInfoHash, selections) {