I run clang-tidy locally in vim and it caught that in this function we take an input parameter by copy and then return a reference to its memory (which is bad because the temporary object will be destroyed when it goes out of scope.

We should fix this.