Ticket #8786 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

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:1 Changed 7 years ago by Arturs Bekasovs

  • Status changed from new to inprogress

Refs #8786. Fix the warnings.

Changeset: 3e5cbda9b97a26c6268eb63fd02efb2067ed65ce

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

comment:8 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 9630

Note: See TracTickets for help on using tickets.