refactor: make setActiveSub parameter optional

Author: johnsoncodehk

src/index.ts

@@ -71,7 +71,7 @@ export function getActiveSub(): ReactiveNode | undefined {
 	return activeSub;
 }
 
-export function setActiveSub(sub: ReactiveNode | undefined) {
+export function setActiveSub(sub?: ReactiveNode) {
 	const prevSub = activeSub;
 	activeSub = sub;
 	return prevSub;

tests/effect.spec.ts

@@ -184,7 +184,7 @@ test('should duplicate subscribers do not affect the notify order', () => {
 
 	effect(() => {
 		order.push('a');
-		const currentSub = setActiveSub(undefined);
+		const currentSub = setActiveSub();
 		const isOne = src2() === 1;
 		setActiveSub(currentSub);
 		if (isOne) {

tests/issue_48.spec.ts

@@ -88,7 +88,7 @@ function reaction<T>(
 }
 
 function untracked<T>(callback: () => T): T {
-	const currentSub = setActiveSub(undefined);
+	const currentSub = setActiveSub();
 	try {
 		return callback();
 	} finally {

tests/untrack.spec.ts

@@ -7,7 +7,7 @@ test('should pause tracking in computed', () => {
 	let computedTriggerTimes = 0;
 	const c = computed(() => {
 		computedTriggerTimes++;
-		const currentSub = setActiveSub(undefined);
+		const currentSub = setActiveSub();
 		const value = src();
 		setActiveSub(currentSub);
 		return value;
@@ -29,7 +29,7 @@ test('should pause tracking in effect', () => {
 	effect(() => {
 		effectTriggerTimes++;
 		if (is()) {
-			const currentSub = setActiveSub(undefined);
+			const currentSub = setActiveSub();
 			src();
 			setActiveSub(currentSub);
 		}
@@ -63,7 +63,7 @@ test('should pause tracking in effect scope', () => {
 	effectScope(() => {
 		effect(() => {
 			effectTriggerTimes++;
-			const currentSub = setActiveSub(undefined);
+			const currentSub = setActiveSub();
 			src();
 			setActiveSub(currentSub);
 		});