delete pre_effect

pull/10960/head
Rich Harris 6 months ago
parent c89b5e6bd1
commit 4abb7482d5

@ -233,7 +233,7 @@ function setup_select_synchronization(value_binding, context) {
context.state.init.push(
b.stmt(
b.call(
'$.pre_effect',
'$.render_effect',
b.thunk(
b.block([
b.stmt(

@ -128,7 +128,7 @@ export function user_pre_effect(fn) {
);
}
return pre_effect(fn);
return render_effect(fn);
}
/**
@ -162,7 +162,7 @@ export function legacy_pre_effect(deps, fn) {
current_component_context
);
const token = {};
return pre_effect(() => {
return render_effect(() => {
deps();
if (component_context.l1.includes(token)) {
return;
@ -190,14 +190,6 @@ export function legacy_pre_effect_reset() {
* @param {() => void | (() => void)} fn
* @returns {import('#client').Effect}
*/
export function pre_effect(fn) {
return create_effect(RENDER_EFFECT, fn, true);
}
/**
* @param {(() => void)} fn
* @returns {import('#client').Effect}
*/
export function render_effect(fn) {
return create_effect(RENDER_EFFECT, fn, true);
}

@ -1,4 +1,4 @@
import { pre_effect, effect_root } from '../internal/client/reactivity/effects.js';
import { render_effect, effect_root } from '../internal/client/reactivity/effects.js';
import { flushSync } from '../main/main-client.js';
import { ReactiveMap } from './map.js';
import { assert, test } from 'vitest';
@ -15,15 +15,15 @@ test('map.values()', () => {
const log: any = [];
const cleanup = effect_root(() => {
pre_effect(() => {
render_effect(() => {
log.push(map.size);
});
pre_effect(() => {
render_effect(() => {
log.push(map.has(3));
});
pre_effect(() => {
render_effect(() => {
log.push(Array.from(map.values()));
});
});
@ -51,15 +51,15 @@ test('map.get(...)', () => {
const log: any = [];
const cleanup = effect_root(() => {
pre_effect(() => {
render_effect(() => {
log.push('get 1', map.get(1));
});
pre_effect(() => {
render_effect(() => {
log.push('get 2', map.get(2));
});
pre_effect(() => {
render_effect(() => {
log.push('get 3', map.get(3));
});
});
@ -87,15 +87,15 @@ test('map.has(...)', () => {
const log: any = [];
const cleanup = effect_root(() => {
pre_effect(() => {
render_effect(() => {
log.push('has 1', map.has(1));
});
pre_effect(() => {
render_effect(() => {
log.push('has 2', map.has(2));
});
pre_effect(() => {
render_effect(() => {
log.push('has 3', map.has(3));
});
});
@ -132,7 +132,7 @@ test('map handling of undefined values', () => {
const cleanup = effect_root(() => {
map.set(1, undefined);
pre_effect(() => {
render_effect(() => {
log.push(map.get(1));
});

@ -1,4 +1,4 @@
import { pre_effect, effect_root } from '../internal/client/reactivity/effects.js';
import { render_effect, effect_root } from '../internal/client/reactivity/effects.js';
import { flushSync } from '../main/main-client.js';
import { ReactiveSet } from './set.js';
import { assert, test } from 'vitest';
@ -9,15 +9,15 @@ test('set.values()', () => {
const log: any = [];
const cleanup = effect_root(() => {
pre_effect(() => {
render_effect(() => {
log.push(set.size);
});
pre_effect(() => {
render_effect(() => {
log.push(set.has(3));
});
pre_effect(() => {
render_effect(() => {
log.push(Array.from(set));
});
});
@ -41,15 +41,15 @@ test('set.has(...)', () => {
const log: any = [];
const cleanup = effect_root(() => {
pre_effect(() => {
render_effect(() => {
log.push('has 1', set.has(1));
});
pre_effect(() => {
render_effect(() => {
log.push('has 2', set.has(2));
});
pre_effect(() => {
render_effect(() => {
log.push('has 3', set.has(3));
});
});

Loading…
Cancel
Save