Tóth András

> Well, as far as I can see the only functional change here is that a method that always returned `nil` before now has a `table` return value. This cannot...

> Shall I remove those commits? How? (Bunnybot merged back my old branch after a force push, so I am not sure anymore how things work.) I'd first create a...

> Should there be one usage example (of one tree) or none at all in this pr? Now I'm a bit confused by this. Isn't there some other place that...


> * The builder leaves immediately and aborts the work. +1 > (The last used material must be requested again. Or any, can be the cheapest or ...) This feature...

Is this really a must for v1.2? We've been doing quite well with the health bar marking warships. On the technical side: The Barbarian ship model in the media repo...

My comments from CB didn't get mirrored...

Please note that `…Widelands.appdata.xml` is an autogenerated file. You have to edit `…appdata.xml.stub`. To make the name actually translatable, you have to introduce a new hook in that file, by...

> Please note that `…Widelands.appdata.xml` is an autogenerated file. You have to edit `…appdata.xml.stub`. To make the name actually translatable, you have to introduce a new hook in that file,...

That's OK. It's not urgent, and I have plenty of other things to do too. (both around Widelands and unfortunately also in my private life... :( )