2 ms·
> but because we have this data (file name, etc.) passed back and forth between different layers in the code. we were doing lots of .clone() and db.open() at di
by binhex 8mo ago
> but because we have this data (file name, etc.) passed back and forth between different layers in the code. we were doing lots of .clone() and db.open() at different layers to fetch the same data. The interesting part for me was that, this change reduced our
Reduced your what? The article seems to be cut off.
- taeric 8mo agoI'm assuming it was going to say lines of code. It is highlighting that more was deleted than added.
- larpingscholar 8mo agoI agree. Though the line count is meaningless as the vibe coded diff has pointless formatting and whitespace changes.
- suriya-ganesh 8mo agoI wrote that PR and can guarantee you that it was not vibe coded. the reason why I thought this PR was interesting is that, it only needed ~ 4 lines of real change, everything else was cascades due to that change. and the performance did improve with that "pointless formatting and whitespace changes"
- suriya-ganesh 8mo agoah! sorry. I meant to say, "the interesting part for me was that we removed code to improve performance."