nuisance is hosted by Hepforge, IPPP Durham
close Warning: Can't synchronize with repository "(default)" ("(default)" is not readable or not a Git repository.). Look in the Trac log for more information.

Opened 7 years ago

Last modified 7 years ago

#48 new task

Method arguments should take (const) references where possible

Reported by: lp208 Owned by: somebody
Priority: minor Milestone: Tuning paper, vxry
Component: component1 Version:
Keywords: code maintenance Cc:

Description

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.

Change History (2)

comment:1 Changed 7 years ago by Patrick Stowell

Milestone: → milestone2

comment:2 Changed 7 years ago by Clarence Wret

Milestone: milestone2 → Tuning paper, vxry

Milestone renamed

Note: See TracTickets for help on using tickets.