Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/bug-1535732-fun-with-pointers into lp:widelands

2016-04-29 Thread TiborB
If you as an author do not feel this is useful I will not give my approval here. Of course somebody else can consider this good change and will approve it. -- https://code.launchpad.net/~widelands-dev/widelands/bug-1535732-fun-with-pointers/+merge/292735 Your team Widelands Developers is

[Widelands-dev] [Merge] lp:~widelands-dev/widelands/bug-1531312-harvested-fields into lp:widelands

2016-04-29 Thread noreply
The proposal to merge lp:~widelands-dev/widelands/bug-1531312-harvested-fields into lp:widelands has been updated. Status: Needs review => Merged For more details, see: https://code.launchpad.net/~widelands-dev/widelands/bug-1531312-harvested-fields/+merge/293161 -- Your team Widelands

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/bug-1535732-fun-with-pointers into lp:widelands

2016-04-29 Thread TiborB
I wonder what is the logic behind all these '!= nullptr', what difference does it make? -- https://code.launchpad.net/~widelands-dev/widelands/bug-1535732-fun-with-pointers/+merge/292735 Your team Widelands Developers is subscribed to branch

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/bug-1531312-harvested-fields into lp:widelands

2016-04-29 Thread GunChleoc
@bunnybot merge -- https://code.launchpad.net/~widelands-dev/widelands/bug-1531312-harvested-fields/+merge/293161 Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/bug-1531312-harvested-fields. ___ Mailing list:

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/bug-1531312-harvested-fields into lp:widelands

2016-04-29 Thread GunChleoc
Thannks! bunnybot merge -- https://code.launchpad.net/~widelands-dev/widelands/bug-1531312-harvested-fields/+merge/293161 Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/bug-1531312-harvested-fields. ___ Mailing

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/bug-1531312-harvested-fields into lp:widelands

2016-04-29 Thread TiborB
Review: Approve LGTM -- https://code.launchpad.net/~widelands-dev/widelands/bug-1531312-harvested-fields/+merge/293161 Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/bug-1531312-harvested-fields. ___ Mailing list:

[Widelands-dev] [Merge] lp:~widelands-dev/widelands/bug-1535732-testfix into lp:widelands

2016-04-29 Thread bunnybot
Continuous integration builds have changed state: Travis build 1058. State: passed. Details: https://travis-ci.org/widelands/widelands/builds/125399521. Appveyor build 889. State: success. Details:

[Widelands-dev] [Merge] lp:~widelands-dev/widelands/bug-1535732-testfix into lp:widelands

2016-04-29 Thread bunnybot
Bunnybot encountered an error while working on this merge proposal: The read operation timed out -- https://code.launchpad.net/~widelands-dev/widelands/bug-1535732-testfix/+merge/292734 Your team Widelands Developers is requested to review the proposed merge of

[Widelands-dev] [Merge] lp:~widelands-dev/widelands/bug-1536377-texture-atlas-size into lp:widelands

2016-04-29 Thread GunChleoc
The proposal to merge lp:~widelands-dev/widelands/bug-1536377-texture-atlas-size into lp:widelands has been updated. Status: Needs review => Work in progress For more details, see: https://code.launchpad.net/~widelands-dev/widelands/bug-1536377-texture-atlas-size/+merge/293067 -- Your

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/bug-1536377-texture-atlas-size into lp:widelands

2016-04-29 Thread GunChleoc
That didn't work, back to the drawing board. -- https://code.launchpad.net/~widelands-dev/widelands/bug-1536377-texture-atlas-size/+merge/293067 Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/bug-1536377-texture-atlas-size into