Should these params be taken by value?