Code review comment for lp:~widelands-dev/widelands/bug-1525706-artifacts

Revision history for this message
SirVer (sirver) wrote :

Sorry, cannot keep up with email....

I think this can be a vector and that would probably be better.

> Am 16.02.2016 um 09:48 schrieb GunChleoc <email address hidden>:
>
> I still need help with this before we can go ahead:
>
>> I tried and failed to just iterate over the fields - seems like there is no
>> way to safely get the size of a std::unique_ptr<array[]>.
>>
>> Why is it an array and not a vector anyway?
> --
> https://code.launchpad.net/~widelands-dev/widelands/bug-1525706-artifacts/+merge/285566
> Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/bug-1525706-artifacts.
>
> _______________________________________________
> Mailing list: https://launchpad.net/~widelands-dev
> Post to : <email address hidden>
> Unsubscribe : https://launchpad.net/~widelands-dev
> More help : https://help.launchpad.net/ListHelp

« Back to merge proposal