Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/bug-1687043-multiline-edit into lp:widelands

2017-04-29 Thread GunChleoc
Review: Approve Code LGTM too :) -- https://code.launchpad.net/~widelands-dev/widelands/bug-1687043-multiline-edit/+merge/323428 Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/bug-1687043-multiline-edit. ___ Mailing

[Widelands-dev] [Merge] lp:~widelands-dev/widelands/bug-1687043-multiline-edit into lp:widelands

2017-04-29 Thread bunnybot
Continuous integration builds have changed state: Travis build 2128. State: passed. Details: https://travis-ci.org/widelands/widelands/builds/227161415. Appveyor build 1963. State: success. Details: https://ci.appveyor.com/project/widelands-dev/widelands/build/_widelands_dev_widelands_bug_168704

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/bug-1687043-multiline-edit into lp:widelands

2017-04-29 Thread Klaus Halfmann
Looks fine for me, will compile this and check some Mutiline text fields -- https://code.launchpad.net/~widelands-dev/widelands/bug-1687043-multiline-edit/+merge/323428 Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/bug-1687043-multilin

[Widelands-dev] [Merge] lp:~widelands-dev/widelands/bug-1687043-multiline-edit into lp:widelands

2017-04-29 Thread Notabilis
Notabilis has proposed merging lp:~widelands-dev/widelands/bug-1687043-multiline-edit into lp:widelands. Requested reviews: Widelands Developers (widelands-dev) Related bugs: Bug #1687043 in widelands: "Memory leak in Multilineeditbox" https://bugs.launchpad.net/widelands/+bug/1687043 For