Shouldn't this be a const reference at least?