remove unused second argument to derived

proxied-state-each-blocks
Rich Harris 9 months ago
parent afd2efa2d7
commit 84a30b55cd

@ -1121,11 +1121,10 @@ export function destroy_signal(signal) {
/**
* @template V
* @param {() => V} init
* @param {import('./types.js').EqualsFunctions} [equals]
* @returns {import('./types.js').ComputationSignal<V>}
*/
/*#__NO_SIDE_EFFECTS__*/
export function derived(init, equals) {
export function derived(init) {
const is_unowned = current_effect === null;
const flags = is_unowned ? DERIVED | UNOWNED : DERIVED;
const signal = /** @type {import('./types.js').ComputationSignal<V>} */ (
@ -1133,7 +1132,7 @@ export function derived(init, equals) {
);
signal.i = init;
signal.x = current_component_context;
signal.e = get_equals_method(equals);
signal.e = default_equals;
if (!is_unowned) {
push_reference(/** @type {import('./types.js').EffectSignal} */ (current_effect), signal);
}

@ -112,10 +112,7 @@ describe('signals', () => {
const A = $.source(0);
const B = $.source(0);
const C = $.derived(() => ($.get(A) % 2) + ($.get(B) % 2));
const D = $.derived(
() => numbers.map((i) => i + ($.get(A) % 2) - ($.get(B) % 2)),
(l: number[], r: number[]) => l.length === r.length && l.every((v, i) => v === r[i])
);
const D = $.derived(() => numbers.map((i) => i + ($.get(A) % 2) - ($.get(B) % 2)));
const E = $.derived(() => hard($.get(C) + $.get(A) + $.get(D)[0]!, 'E'));
const F = $.derived(() => hard($.get(D)[0]! && $.get(B), 'F'));
const G = $.derived(() => $.get(C) + ($.get(C) || $.get(E) % 2) + $.get(D)[0]! + $.get(F));

Loading…
Cancel
Save