Frostregen wrote:

minor coding style fix.

Attachments

• patchsetting_tree_growth_v2.diff (9.81 KiB)

This comment was imported from FlySpray: https://bugs.openttd.org/task/1605#comment3119

Frostregen wrote:

again, coding style fix

Attachments

• patchsetting_tree_growth_v3.diff (9.80 KiB)

This comment was imported from FlySpray: https://bugs.openttd.org/task/1605#comment3121

Frostregen wrote:

simplified code. (patchfile is more readable now)

Attachments

• patchsetting_tree_growth_v4.diff (3.65 KiB)

This comment was imported from FlySpray: https://bugs.openttd.org/task/1605#comment3122

Frostregen wrote:

added patchsetting: "Disable Tree grow in scenario editor",
which is a config setting only -> not saved with map.
synced with trunk r11811

Attachments

• patchsetting_tree_growth_v5.diff (4.24 KiB)

This comment was imported from FlySpray: https://bugs.openttd.org/task/1605#comment3222

planetmaker wrote:

updated to r13305

Attachments

• patchsetting_treegrowth_r13305.diff (9.08 KiB)

This comment was imported from FlySpray: https://bugs.openttd.org/task/1605#comment4204

planetmaker wrote:

updated, simplified a lot and fixed coding style (actually by Belugas)

Attachments

• patchsetting_treegrowth_v2_r13435.diff (3.59 KiB)

This comment was imported from FlySpray: https://bugs.openttd.org/task/1605#comment4254

skidd13 wrote:

The main problem with this patch is that some industries depend on tree growth, like lumber mills in tropic. And additionaly such industries can be defined via GRF, so the patch breaks current functionality. At it's current design the patch wont make it into trunk.

This comment was imported from FlySpray: https://bugs.openttd.org/task/1605#comment4262

Belugas wrote:

I can confirm that.
You can use it for patch packs, it's ok since it's "network" friendly, but it willnot make it in trunk because of the above reasons. And Planetmaker himself is aware of the reasons.

This comment was imported from FlySpray: https://bugs.openttd.org/task/1605#comment4263

Frostregen wrote:

In theory we could check for a nearby industry which depends on tree-growth, and allow those trees to grow anyway.
In practice this check could be quite a slowdown with many trees.

Maybe the other way round?
Industries let the trees grow around them? (Only if this patchsetting is enabled)

This comment was imported from FlySpray: https://bugs.openttd.org/task/1605#comment4276

SmatZ closed the ticket.

Reason for closing: Won't implement

You can switch to invisible trees. When trees don't grow, there won't be any if you demolish them. Also, it's not realistic ;-)

This comment was imported from FlySpray: https://bugs.openttd.org/task/1605