widelands-dev team mailing list archive

Thread
Date

Re: [Merge] lp:~widelands-dev/widelands/bug-986611-cppcheck_performance-ui into lp:widelands

To: mp+300992@xxxxxxxxxxxxxxxxxx
From: GunChleoc <fios@xxxxxxxxxxxxxxxxxxx>
Date: Mon, 25 Jul 2016 08:43:05 -0000
In-reply-to: <20160724115622.22935.14471.launchpad@ackee.canonical.com>
Reply-to: mp+300992@xxxxxxxxxxxxxxxxxx
Sender: bounces@xxxxxxxxxxxxx

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.

References

[Merge] lp:~widelands-dev/widelands/bug-986611-cppcheck_performance-ui into lp:widelands
From: GunChleoc, 2016-07-24