Skip to content

Commit 590756c

Browse files
Refactored;
1 parent a0a7144 commit 590756c

File tree

1 file changed

+4
-5
lines changed

1 file changed

+4
-5
lines changed

lib/use-async-effect.js

+4-5
Original file line numberDiff line numberDiff line change
@@ -10,15 +10,14 @@ const isGeneratorFn = (thing) => typeof thing === 'function' &&
1010
thing.constructor.name === 'GeneratorFunction';
1111

1212
const useAsyncEffect = (generator, deps) => {
13-
let _promise;
13+
let promise;
1414

1515
useEffect(() => {
1616
if (!isGeneratorFn(generator)) {
1717
throw TypeError('useAsyncEffect requires a generator as the first argument');
1818
}
1919

20-
const promise = CPromise.resolveGenerator(generator, {resolveSignatures: true});
21-
_promise = promise;
20+
promise = CPromise.resolveGenerator(generator, {resolveSignatures: true});
2221

2322
let cb;
2423

@@ -31,13 +30,13 @@ const useAsyncEffect = (generator, deps) => {
3130

3231
return () => {
3332
promise.cancel(E_REASON_UNMOUNTED);
34-
_promise = null;
33+
promise = null;
3534
cb && cb();
3635
}
3736
}, deps);
3837

3938
return [
40-
(reason) => !!_promise && _promise.cancel(reason)
39+
(reason) => !!promise && promise.cancel(reason)
4140
]
4241
}
4342

0 commit comments

Comments
 (0)