I only looked at the diff and the lua-changes look good to me. I also like the extended documentation. I can't say much about the source code change. The code itself looks okay but I have no idea what exactly _() and pgettext_exp() are doing. -- https://code.launchpad.net/~widelands-dev/widelands/disambiguate_production_program_names/+merge/316933 Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/disambiguate_production_program_names into lp:widelands.
_______________________________________________ Mailing list: https://launchpad.net/~widelands-dev Post to : widelands-dev@lists.launchpad.net Unsubscribe : https://launchpad.net/~widelands-dev More help : https://help.launchpad.net/ListHelp
