Details
Notice things are deduplicated and fixed.
Diff Detail
- Repository
- rP 0 A.D. Public Repository
- Branch
- master_up
- Lint
Lint OK - Unit
No Unit Test Coverage - Build Status
Buildable 9163 Build 15045: Vulcan Build Jenkins Build 15044: Vulcan Build (Windows) Jenkins Build 15043: 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/vs2015-differential/34/display/redirect
Build failure - The Moirai have given mortals hearts that can endure.
Linter detected issues: Executing section Source... source/simulation2/components/ICmpRangeManager.h | 69| class·ICmpRangeManager·:·public·IComponent | | [MAJOR] CPPCheckBear (syntaxError): | | Code 'classICmpRangeManager:' is invalid C code. Use --std or --language to configure the language. Executing section JS... Executing section cli...
Link to build: https://jenkins.wildfiregames.com/job/docker-differential/543/display/redirect
As far as I know inline doesn't do anything, because the compiler decides to inline according to its own heuristics.
Only effect I found was that gcc shows a compiler warning if it decides to not inline.
Thanks for the fix and duplication removal.
Tested with clang 8.0.1.
source/simulation2/components/ICmpRangeManager.cpp | ||
---|---|---|
37 | Perhaps it should return std::string like the using functions? |
inline can be used to get around the ODR. However it seems an anonymous namespace is safer.
Also return std::string directly.
Successful build - Chance fights ever on the side of the prudent.
Link to build: https://jenkins.wildfiregames.com/job/vs2015-differential/35/display/redirect
Build failure - The Moirai have given mortals hearts that can endure.
Linter detected issues: Executing section Source... source/simulation2/components/ICmpRangeManager.h | 69| class·ICmpRangeManager·:·public·IComponent | | [MAJOR] CPPCheckBear (syntaxError): | | Code 'classICmpRangeManager:' is invalid C code. Use --std or --language to configure the language. Executing section JS... Executing section cli...
Link to build: https://jenkins.wildfiregames.com/job/docker-differential/544/display/redirect