some stylistic changes

zloirock · Mar 31, 2019 · 7c4a18f · 7c4a18f
1 parent e3b99db
commit 7c4a18f
Show file tree

Hide file tree

Showing 5 changed files with 15 additions and 13 deletions.
diff --git a/packages/core-js/internals/perform.js b/packages/core-js/internals/perform.js
@@ -1,7 +1,7 @@
 module.exports = function (exec) {
   try {
-    return { e: false, v: exec() };
+    return { error: false, value: exec() };
   } catch (error) {
-    return { e: true, v: error };
+    return { error: true, value: error };
   }
 };
diff --git a/packages/core-js/modules/es.promise.js b/packages/core-js/modules/es.promise.js
@@ -144,8 +144,8 @@ var onUnhandled = function (promise, state) {
       });
       // Browsers should not trigger `rejectionHandled` event if it was handled here, NodeJS - should
       state.rejection = IS_NODE || isUnhandled(state) ? UNHANDLED : HANDLED;
+      if (result.error) throw result.value;
     }
-    if (IS_UNHANDLED && result.e) throw result.v;
   });
 };
 
@@ -326,7 +326,7 @@ $export({ target: PROMISE, stat: true, forced: INCORRECT_ITERATION }, {
       });
       --remaining || resolve(values);
     });
-    if (result.e) reject(result.v);
+    if (result.error) reject(result.value);
     return capability.promise;
   },
   // `Promise.race` method
@@ -340,7 +340,7 @@ $export({ target: PROMISE, stat: true, forced: INCORRECT_ITERATION }, {
         C.resolve(promise).then(capability.resolve, reject);
       });
     });
-    if (result.e) reject(result.v);
+    if (result.error) reject(result.value);
     return capability.promise;
   }
 });
diff --git a/packages/core-js/modules/esnext.promise.all-settled.js b/packages/core-js/modules/esnext.promise.all-settled.js
@@ -34,7 +34,7 @@ require('../internals/export')({ target: 'Promise', stat: true }, {
       });
       --remaining || resolve(values);
     });
-    if (result.e) reject(result.v);
+    if (result.error) reject(result.value);
     return capability.promise;
   }
 });
diff --git a/packages/core-js/modules/esnext.promise.any.js b/packages/core-js/modules/esnext.promise.any.js
@@ -36,7 +36,7 @@ require('../internals/export')({ target: 'Promise', stat: true }, {
       });
       --remaining || reject(new (getBuiltIn('AggregateError'))(errors, PROMISE_ANY_ERROR));
     });
-    if (result.e) reject(result.v);
+    if (result.error) reject(result.value);
     return capability.promise;
   }
 });
diff --git a/packages/core-js/modules/esnext.promise.try.js b/packages/core-js/modules/esnext.promise.try.js
@@ -4,9 +4,11 @@
 var newPromiseCapabilityModule = require('../internals/new-promise-capability');
 var perform = require('../internals/perform');
 
-require('../internals/export')({ target: 'Promise', stat: true }, { 'try': function (callbackfn) {
-  var promiseCapability = newPromiseCapabilityModule.f(this);
-  var result = perform(callbackfn);
-  (result.e ? promiseCapability.reject : promiseCapability.resolve)(result.v);
-  return promiseCapability.promise;
-} });
+require('../internals/export')({ target: 'Promise', stat: true }, {
+  'try': function (callbackfn) {
+    var promiseCapability = newPromiseCapabilityModule.f(this);
+    var result = perform(callbackfn);
+    (result.error ? promiseCapability.reject : promiseCapability.resolve)(result.value);
+    return promiseCapability.promise;
+  }
+});