From Dev.to
Your Refactor Copies a List. The Caller Still Holds the Old One.
In-place mutation is the quietest part of a module's contract. Most characterization tests assert what a function returns. Almost none assert that the caller's list changed underneath. Replace rows.so...