Details
Diff Detail
- Repository
- rP 0 A.D. Public Repository
- Branch
- waterFix
- Lint
Lint OK - Unit
No Unit Test Coverage - Build Status
Buildable 6613 Build 10908: Vulcan Build Jenkins Build 10907: arc lint + arc unit
Event Timeline
Successful build - Chance fights ever on the side of the prudent.
Link to build: https://jenkins.wildfiregames.com/job/differential/861/
Add some comments. This is actually quite sane and works correctly (and fixes an issue on Polynesia) so I'll commit it soon-ish.
| source/renderer/WaterManager.cpp | ||
|---|---|---|
| 1 | Year. | |
| 1021 | Can that happen ? | |
| 1035–1036 | 1.0f ? | |
| 1038 | Maybe it should be declared out of the scope ? | |
| 1041 | Put comment on top ? | |
| 1048 | spaces between operators. | |
| 1049 | static_cast Also can't you just use a normal a size_t ? size_t m_MapSize; | |
| 1055 | spaces between operators. | |
| 1056 | Same here. | |
| 1061 | spaces between operators. | |
| 1063 | spaces between operators. | |
| 1064 | Same here. | |
| 1068 | can't this be 'simplified' using a ternary and a 'end' variable ? | |
| 1075 | Comments start with caps. | |
| 1087–1108 | missing spaces ? | |
| 1100 | 1.0f ? | |
| 1103 | spaces between operators. | |
| 1107 | Missing braces. | |
| 1112–1113 | static_cast / why the cast ? | |
Successful build - Chance fights ever on the side of the prudent.
Link to build: https://jenkins.wildfiregames.com/job/differential/1493/display/redirect
Build failure - The Moirai have given mortals hearts that can endure.
Link to build: https://jenkins.wildfiregames.com/job/docker-differential/382/display/redirect
| source/renderer/WaterManager.cpp | ||
|---|---|---|
| 1066 | comments start with capital :) | |
| source/renderer/WaterManager.cpp | ||
|---|---|---|
| 1021 | Yes before init. | |
Successful build - Chance fights ever on the side of the prudent.
Link to build: https://jenkins.wildfiregames.com/job/docker-differential/387/display/redirect
Ran some profiling, and I'm not actually sure this speeds things up. But it's still better behaviour as there are some artefacts with the existing code on e.g. Polynesia near the edges, and it's more readable imo so I'll commit anyways.