Hacker News new | ask | show | jobs
by binhex 142 days 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.

2 comments

I'm assuming it was going to say lines of code. It is highlighting that more was deleted than added.
I agree. Though the line count is meaningless as the vibe coded diff has pointless formatting and whitespace changes.
I 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"

ah! sorry. I meant to say, "the interesting part for me was that we removed code to improve performance."