Feature Request #2332

Engine overrides (action0 feature 8 prop 11)

Added by Hirundo almost 9 years ago. Updated over 8 years ago.

Status:ClosedStart date:2011-02-16
Priority:NormalDue date:
Assignee:-% Done:

100%

Category:-
Target version:-

Description

Suggested syntax:

override(target[, source])

if not set specifically, source is set to the grfid of the current grf.

Associated revisions

Revision 1297:bb3505472aa0
Added by yexo over 8 years ago

Feature #2332: engine overrides (action0 general variables prop 0x11)

Revision 1297:bb3505472aa0
Added by yexo over 8 years ago

Feature #2332: engine overrides (action0 general variables prop 0x11)

Revision 1372:c7d59ce6bf9b
Added by Hirundo over 8 years ago

Doc: Improve documentation of the engine_override command (#2332).

Revision 1372:c7d59ce6bf9b
Added by Hirundo over 8 years ago

Doc: Improve documentation of the engine_override command (#2332).

Revision 1373:f5c4560bdeac
Added by Hirundo over 8 years ago

Add #2332: Regression test for engine override.

Revision 1373:f5c4560bdeac
Added by Hirundo over 8 years ago

Add #2332: Regression test for engine override.

Revision 1374:f11d7dc8af6d
Added by Hirundo over 8 years ago

Fix #2332: Order of parameters of engine_override(..) was mis-documented.

Revision 1374:f11d7dc8af6d
Added by Hirundo over 8 years ago

Fix #2332: Order of parameters of engine_override(..) was mis-documented.

History

#1 Updated by Ammler over 8 years ago

you also migh need to check, if source is already used.

also is it possible to override 2 different newgrfs with same source?

Edit: it could also be part of context grf

#2 Updated by Ammler over 8 years ago

just learned, you might not know what I mean with "context" ;-)

I would make the override properity as part of the block grf:

grf { override: <target>; }

#3 Updated by yexo over 8 years ago

Syntax:
engine_override(source, target);
engine_override(target);

Still needs documentation / regression test.

#4 Updated by planetmaker over 8 years ago

  • % Done changed from 0 to 50

Documentation in r1354. Regression still needed.

#5 Updated by Hirundo over 8 years ago

  • Status changed from New to Closed
  • % Done changed from 50 to 100

Applied in changeset f11d7dc8af6d.

Also available in: Atom PDF