added comments describing why we use get_registered_params before notifying read methods

Author: FoHoOV

packages/svelte/src/reactivity/map.js

@@ -20,13 +20,15 @@ export const ReactiveMap = make_reactive(Map, {
 			}
 
 			options.get_registered_params('has')?.forEach((value, param) => {
+				// because we don't want to notify `has` for items that are currently not in the map
 				if (!options.value.has(param)) {
 					return;
 				}
 				options.notify_read_properties(['has'], param);
 			});
 
 			options.get_registered_params('get')?.forEach((value, param) => {
+				// because we don't want to notify `get` for items that are currently not in the map
 				if (!options.value.has(param)) {
 					return;
 				}

packages/svelte/src/reactivity/set.js

@@ -16,6 +16,7 @@ export const ReactiveSet = make_reactive(Set, {
 				return false;
 			}
 			options.get_registered_params('has')?.forEach((value, param) => {
+				// because we don't want to notify `has` for items that are currently not in the set
 				if (!options.value.has(param)) {
 					return;
 				}