The vector thing is actually quite counterintuitive - one would expect a reference not to copy all the elements *headdesk*. I'm sure there is some reason for that design, do you know why they did it that way?
I have added your explanation to my notes :) @bunnybot merge -- https://code.launchpad.net/~widelands-dev/widelands/bug-986611-cppcheck_performance-ui/+merge/300992 Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/bug-986611-cppcheck_performance-ui. _______________________________________________ Mailing list: https://launchpad.net/~widelands-dev Post to : [email protected] Unsubscribe : https://launchpad.net/~widelands-dev More help : https://help.launchpad.net/ListHelp

