Michael Dellanoce

Results 27 comments of Michael Dellanoce

i updated my PR #1300 with another potential solution

i updated #1300 with a benchmark for the 2nd style of mutation mentioned above

@wfk007 seems a little bit slower than rebuilding the list for that specific case: However, yes, I can see that the other cases were less impacted, and overall it is...

@wfk007 it looks like the first time i ran the benchmark I had this other fix applied: https://github.com/rrweb-io/rrweb/pull/1294, which gives me the 170ish result. If i do the same with...

@wfk007 I converted this PR to "draft" and added 2 commits that I've been working on. 1. On the recording side, I changed the serialization process for added nodes so...

Added another benchmark for node insertion order. This one does not _currently_ perform poorly, but it does perform poorly with the fix from #1302 so this is just meant to...

@wfk007 yes, 2 was my major concern. I agree with @colingm that 4 might be the correct trade-off. I've had the opportunity to see rrweb performing side-by-side with some of...