Page MenuHomeWildfire Games

Queries water manager only if a model should account it
ClosedPublic

Authored by vladislavbelov on Sep 7 2023, 7:33 PM.

Details

Reviewers
phosit
Group Reviewers
Restricted Owners Package(Owns No Changed Paths)
Commits
rP27833: Queries water manager only if a model should account it.
Summary

Subj.

Test Plan
  1. Apply the patch and compile the game
  2. Make sure floating and non-floating models behave like without the patch

Diff Detail

Repository
rP 0 A.D. Public Repository
Lint
Lint Skipped
SeverityLocationCodeMessage
Errorsource/graphics/Model.cpp:1LICENCE YEAR1Inaccurate Copyright Year
Unit
Unit Tests Skipped
Build Status
Buildable 22298
Build 54519: Vulcan BuildJenkins
Build 54518: Vulcan Build (macOS)Jenkins
Build 54517: Vulcan Build (Windows)Jenkins

Event Timeline

vladislavbelov created this revision.Sep 7 2023, 7:33 PM
Vulcan added a comment.Sep 7 2023, 7:34 PM

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

Link to build: https://jenkins.wildfiregames.com/job/macos-differential/7257/display/redirect

Vulcan added a comment.Sep 7 2023, 7:45 PM

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

Link to build: https://jenkins.wildfiregames.com/job/vs2015-differential/8346/display/redirect

vladislavbelov requested review of this revision.Sep 7 2023, 8:19 PM
phosit added a subscriber: phosit.EditedSep 9 2023, 6:57 PM

When testing with my mod: when the water rizes the ships get destroied. That shouldn't be the case but it's the same behavior as before the diff.
When testing with extinct vulcano: there are a lot of errors. Again the same behavior as before the diff.

phosit accepted this revision.Sep 10 2023, 3:20 PM

I can't reproduce the errors on extinct volcano anymore. And it's not related.

This revision is now accepted and ready to land.Sep 10 2023, 3:20 PM
This revision was landed with ongoing or failed builds.Sep 10 2023, 8:02 PM
This revision was automatically updated to reflect the committed changes.