Method arguments should take (const) references where possible
|Reported by:||lp208||Owned by:||somebody|
|Priority:||minor||Milestone:||Tuning paper, vxry|
There are a very large number of needless string copies throughout the codebase that would be gotten rid of by using updating functions and methods to take const string references where possible (almost everywhere).
Probably similarly with any passed vectors/lists, they should be passed as references unless you really mean not to, const where possible.