Page MenuHomeWildfire Games

Remove legacy trigger handling in map reader
AbandonedPublic

Authored by wraitii on May 19 2018, 4:34 PM.

Details

Reviewers
None
Group Reviewers
Restricted Owners Package(Owns No Changed Paths)
Summary

From leper's inevitable.zip

Test Plan

Compile.

Diff Detail

Repository
rP 0 A.D. Public Repository
Branch
D1508_map_triggers
Lint
Lint OK
Unit
No Unit Test Coverage
Build Status
Buildable 6614
Build 10910: Vulcan BuildJenkins
Build 10909: arc lint + arc unit

Event Timeline

wraitii created this revision.May 19 2018, 4:34 PM
Owners added subscribers: Restricted Owners Package, Restricted Owners Package.May 19 2018, 4:34 PM
Vulcan added a subscriber: Vulcan.May 19 2018, 4:54 PM

Successful build - Chance fights ever on the side of the prudent.

Link to build: https://jenkins.wildfiregames.com/job/differential/521/display/redirect

vladislavbelov added inline comments.
source/graphics/MapReader.cpp
921

Declaration of this function is not removed.

wraitii updated this revision to Diff 7148.Dec 30 2018, 5:50 PM

Updated with more deleted

Build failure - The Moirai have given mortals hearts that can endure.

Link to build: https://jenkins.wildfiregames.com/job/differential/858/

wraitii updated this revision to Diff 7152.Dec 30 2018, 6:31 PM

Obviously didn't run the tests

Successful build - Chance fights ever on the side of the prudent.

Link to build: https://jenkins.wildfiregames.com/job/differential/862/

wraitii abandoned this revision.May 5 2019, 9:47 AM

Err, as leper said on that

For this diff I'd like a bump of the map version, apart from that it looks good (and more complete than some diff I had locally). I did somewhat finish up the patch about adding gaia to the player array in maps, so it might be nice to get these two in consecutively, so the map version bump is only done once and handles both of these.

And indeed this is less complete than D642, so I'll abandon this.