Add: consist_current_power in varact2 for trains#419
Add: consist_current_power in varact2 for trains#419WolfieFox wants to merge 1 commit intoOpenTTD:masterfrom
Conversation
|
|
|
Valid, renamed the var to match. |
|
Needs updating to squash the fixup commit, and rebasing so the Black CI check passes. |
7234c59 to
99c42d4
Compare
|
Commit name doesn't match the format (but the PR title is correct). 🙂 |
Damnit, the rebase command prompted me three times and I didn't even parse it the second time. One moment... |
OpenTTD has had these vars exposed since r3594 so I think we should be good to go on that.
99c42d4 to
1ee7451
Compare
2TallTyler
left a comment
There was a problem hiding this comment.
This variable is not published in NewGRF specs, but I found it. 🙂
For others' reference: the source code exposes it in vehicle specific properties -> VEH_TRAIN -> case 0x74 (offset by 0x80):
https://github.com/OpenTTD/OpenTTD/blob/master/src/newgrf_engine.cpp#L924
If you're willing, this could use adding to the specs.
https://newgrf-specs.tt-wiki.net/wiki/VariationalAction2/Vehicles
If/when this is merged, it will also need to be added to the NML specs.
https://newgrf-specs.tt-wiki.net/wiki/NML:Vehicles
OpenTTD has had these vars exposed since r3594 (Feb 2006) so I think we should be good to go on using them