3 ms·
Looks cool. It should be straightforward to make it avoid cycles itself, and default to infinite depth, which feels like a better default than 5.
by pbiggar 14y ago
Looks cool. It should be straightforward to make it avoid cycles itself, and default to infinite depth, which feels like a better default than 5.
- begriffs 14y agoActually it does mark visited objects to avoid cycles in its search. The default depth limit prevents it from taking too long on huge dom objects, but the choice of an arbitrary depth limit (of 5) does seem kind of gross. I'm open to suggestions for improvement.
- pbiggar 14y agoOh cool. The readme (at the very bottom) makes it appear like it doesn't mark visited objects.