Ticket #8786 (closed: fixed)
Fix Doxygen warnings
Reported by: | Arturs Bekasovs | Owned by: | Arturs Bekasovs |
---|---|---|---|
Priority: | minor | Milestone: | Release 3.1 |
Component: | Muon | Keywords: | |
Cc: | Blocked By: | ||
Blocking: | Tester: | Jay Rainey |
Description (last modified by Arturs Bekasovs) (diff)
There are a few Doxygen warnings as reported here:
http://download.mantidproject.org/jenkins/view/Static%20Analysis/job/is_doxygen_develop/lastBuild
Should fix them.
Change History
comment:2 Changed 7 years ago by Arturs Bekasovs
- Component changed from Muon to Framework
- Description modified (diff)
- Summary changed from Fix Doxygen warnings in MuonAnalysis source to Fix Doxygen warnings
Might as well fix a few remaining warnings.
comment:3 Changed 7 years ago by Arturs Bekasovs
- Component changed from Framework to Muon
Having actually looked at the remaining warnings, I am not that confident that I will get the right meaning of some of these undocumented parameters. So better leave them with their authors.
comment:4 Changed 7 years ago by Arturs Bekasovs
- Status changed from inprogress to verify
- Resolution set to fixed
Tester:
Just go to http://download.mantidproject.org/jenkins/view/Static%20Analysis/job/is_doxygen_develop/lastBuild and verify that MuonAnalysis.cpp (or any other MuonAnalysis related file) doesn't have any Doxygen warnings anymore.
comment:5 Changed 7 years ago by Jay Rainey
- Status changed from verify to verifying
- Tester set to Jay Rainey
comment:6 Changed 7 years ago by Jay Rainey
- Status changed from verifying to closed
I checked the doxygen build from yesterday morning. There was four warnings for MuonAnalysis. I then checked a recent doxygen build, and these warnings have been addressed by the change in comment:1.
comment:7 Changed 7 years ago by Jay Rainey
Merge remote-tracking branch 'origin/bugfix/8786_muon_doxygen_warnings'
Full changeset: d1079515f6b528a0dec5d84ac7c6569a6a3d8a99
Refs #8786. Fix the warnings.
Changeset: 3e5cbda9b97a26c6268eb63fd02efb2067ed65ce