Skip to content

Commit e791b53

Browse files
committed
tidy up
1 parent 5df0149 commit e791b53

File tree

2 files changed

+12
-14
lines changed

2 files changed

+12
-14
lines changed

packages/svelte/src/internal/client/reactivity/effects.js

Lines changed: 10 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ import { remove } from '../dom/reconciler.js';
3939
*/
4040
function create_effect(type, fn, sync, init = true) {
4141
/** @type {import('#client').Effect} */
42-
const signal = {
42+
const effect = {
4343
parent: current_effect,
4444
dom: null,
4545
deps: null,
@@ -55,34 +55,35 @@ function create_effect(type, fn, sync, init = true) {
5555
};
5656

5757
if (current_effect !== null) {
58-
signal.l = current_effect.l + 1;
58+
effect.l = current_effect.l + 1;
5959
}
6060

6161
if (current_reaction !== null) {
6262
if (current_reaction.effects === null) {
63-
current_reaction.effects = [signal];
63+
current_reaction.effects = [effect];
6464
} else {
65-
current_reaction.effects.push(signal);
65+
current_reaction.effects.push(effect);
6666
}
6767
}
6868

6969
if (init) {
7070
if (sync) {
7171
const previously_flushing_effect = is_flushing_effect;
72+
7273
try {
7374
set_is_flushing_effect(true);
74-
execute_effect(signal);
75-
set_signal_status(signal, CLEAN);
76-
signal.f |= EFFECT_RAN;
75+
execute_effect(effect);
76+
set_signal_status(effect, CLEAN);
77+
effect.f |= EFFECT_RAN;
7778
} finally {
7879
set_is_flushing_effect(previously_flushing_effect);
7980
}
8081
} else {
81-
schedule_effect(signal);
82+
schedule_effect(effect);
8283
}
8384
}
8485

85-
return signal;
86+
return effect;
8687
}
8788

8889
/**

packages/svelte/src/internal/client/runtime.js

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -216,9 +216,6 @@ export function check_dirtiness(reaction) {
216216
* @returns {V}
217217
*/
218218
export function execute_reaction_fn(signal) {
219-
const fn = signal.fn;
220-
const flags = signal.f;
221-
222219
const previous_dependencies = current_dependencies;
223220
const previous_dependencies_index = current_dependencies_index;
224221
const previous_untracked_writes = current_untracked_writes;
@@ -230,11 +227,11 @@ export function execute_reaction_fn(signal) {
230227
current_dependencies_index = 0;
231228
current_untracked_writes = null;
232229
current_reaction = signal;
233-
current_skip_reaction = !is_flushing_effect && (flags & UNOWNED) !== 0;
230+
current_skip_reaction = !is_flushing_effect && (signal.f & UNOWNED) !== 0;
234231
current_untracking = false;
235232

236233
try {
237-
let res = fn();
234+
let res = signal.fn();
238235
let dependencies = /** @type {import('./types.js').Value<unknown>[]} **/ (signal.deps);
239236
if (current_dependencies !== null) {
240237
let i;

0 commit comments

Comments
 (0)