-
Details
Details
Diff Detail
Diff Detail
- Repository
- rP 0 A.D. Public Repository
- Branch
- upstream
- Lint
Lint Skipped - Unit
Unit Tests Skipped - Build Status
Buildable 77 Build 129: Vulcan Build Jenkins Build 128: arc lint + arc unit
Event Timeline
Comment Actions
Build is green
Updating workspaces. Build (release)... ../../../source/gui/CChart.cpp:38:41: warning: unused parameter ‘Message’ [-Wunused-parameter] void CChart::HandleMessage(SGUIMessage& Message) ^ ../../../source/lib/tex/tex_png.cpp: In member function ‘virtual Status TexCodecPng::encode(Tex*, DynArray*) const’: ../../../source/lib/tex/tex_png.cpp:309:9: warning: variable ‘ret’ might be clobbered by ‘longjmp’ or ‘vfork’ [-Wclobbered] Status ret = ERR::FAIL; ^ Build (debug)... ../../../source/gui/CChart.cpp:38:41: warning: unused parameter ‘Message’ [-Wunused-parameter] void CChart::HandleMessage(SGUIMessage& Message) ^ Running debug tests... Running cxxtest tests (302 tests)..............................................................................................................................................................................................................................................................................................................OK!
http://jw:8080/job/phabricator/61/ for more details.
Comment Actions
(Didn't we intend to attempt to avoid or complete TODOs before committing? Also it is nice to have reviewers mentioned in the commit message)
Comment Actions
(Probably, but I asked someone to add what this TODO refers to quite some time ago and the result was that I can still add this now. Also I'm not going to fix this, so well. Not sure having the reviewers listed for this commit is useful; might be nice to have just reviewed by and differential revision added to the commit message.)