Make ParamSet::find_xxx() take the name as a &str
abusch opened this issue · 1 comments
abusch commented
I noticed that the ParamSet::find_xxx()
methods take the name
argument as a String
. I think it should be a &str
instead as most of the time we pass in a string literal. This would make its use more ergonomic, and avoid lots of unnecessary allocations.
wahn commented
Excellent. Sounds good to me.