Morloth opened the ticket and wrote:

These patches make most methods found in the NoAI API static as they should be. There is no reason why the member functions should be non-static. It makes calling these functions more easy as you can now call:

AICargo.GetCargoLabel(id);

Instead of:
local cargoAI = AICargo();
cargoAI.GetCargoLabel(id);

I've made a different patch file for every class that needs to be modified in the NoAI API. Some might also include changes to the cpp file, as some would contain calls like: 'this->DoCommand(...)'; These are now replaced by 'DoCommand(...);' which does the same, only removes the dependency of an actual class instance.

- Bram

Attachments

• make_static_airport_12185.patch (3.01 KiB)
• make_static_cargo_12185.patch (3.26 KiB)
• make_static_company_12185.patch (9.37 KiB)
• make_static_industry_12185.patch (3.33 KiB)
• make_static_map_12185.patch (1.44 KiB)
• make_static_marine_12185.patch (7.98 KiB)
• make_static_order_12185.patch (10.52 KiB)
• make_static_road_12185.patch (13.85 KiB)
• make_static_sign_12185.patch (4.08 KiB)
• make_static_tile_12185.patch (2.22 KiB)
• make_static_town_12185.patch (1.70 KiB)
• make_static_vehicle_12185.patch (14.71 KiB)

Reported version: 0.5.3
Operating system: All

This issue was imported from FlySpray: https://bugs.openttd.org/task/1785