Ticket #9341 (closed: fixed)

Opened 6 years ago

Last modified 5 years ago

Clear doxygen and cppcheck warnings from master

Reported by: Russell Taylor Owned by: Russell Taylor
Priority: trivial Milestone: Release 3.2
Component: Documentation Keywords:
Cc: Blocked By:
Blocking: Tester: Peter Peterson

Description

If the plan to give static analysis jobs more prominence is going to be useful, we need to be clear the warnings in the first place. This ticket is to fix those that have already made it onto master when people weren't paying enough attention.

In conjunction with this, any we need to make people fix any that they've introduced to the develop branch.

Change History

comment:1 Changed 6 years ago by Russell Taylor

  • Status changed from new to assigned

comment:2 Changed 6 years ago by Russell Taylor

  • Status changed from assigned to inprogress

Re #9341. Fix doxygen warnings.

Changeset: d627271379e5b646e4e36fa75962db3ed5a52b6d

comment:3 Changed 6 years ago by Russell Taylor

Re #9341. Fix typo to eliminate doxygen warnings.

Changeset: 1c4ed16486e24c038015946d28e68f86ddb6ee00

comment:4 Changed 6 years ago by Russell Taylor

Re #9341. Reinstate start point for doxygen exclusion zone.

Changeset: cf2b470d73472bda5a6cb1c886fed2e0d7ff1b94

comment:5 Changed 6 years ago by Russell Taylor

Re #9341. Add missing parameter documentation.

Changeset: f2f7b46179bd1358dc9067e1a60c573f5a380da4

comment:6 Changed 6 years ago by Russell Taylor

Re #9341. Document new(er) parameter.

Changeset: a3e785c33c0cc155b172b7a7efa40ad31c6a21ff

comment:7 Changed 6 years ago by Russell Taylor

Re #9341. Correct type that led to warning from doxygen.

Changeset: a2e3ef67ab5168264090a11270cf7edb60283514

comment:8 Changed 6 years ago by Russell Taylor

Re #9341. Clear doxygen warnings

Fake parameter documentation which isn't great but is consistent with what else is there for this function.

Changeset: 8e0bf8ba396b0f7ecaf0d4ee80f31a630c3a3c0f

comment:9 Changed 6 years ago by Russell Taylor

Re #9341. Don't document parameter that doesn't exist.

Changeset: 25cc2acbce73a772c22a8f55f55e34720af4a616

comment:10 Changed 6 years ago by Russell Taylor

Re #9341. Remove unused file.

Changeset: e1378ec293ff421dc7e4522a28b9c73920e8c150

comment:11 Changed 6 years ago by Russell Taylor

Re #9341. Add missing documentation strings.

Changeset: dde18f9fb8bed07a1750ee3e6df702413e93d6b0

comment:12 Changed 6 years ago by Russell Taylor

Re #9341. Fix doxygen warnings.

Changeset: 85ab981fa061e7f60dc1fabc9494ed634b5a5a5b

comment:13 Changed 6 years ago by Russell Taylor

Re #9341. Fix doxygen warnings.

Changeset: b098b87e1a3b5aada3e2b3e2f1937d169446afbc

comment:14 Changed 6 years ago by Russell Taylor

Re #9341. Do the doxygen documentation right this time.

Changeset: 1f81451a3d51eb28b0ef19ea9f4377088e849d68

comment:15 Changed 6 years ago by Russell Taylor

Re #9341. Reinstate start point for doxygen exclusion zone.

Changeset: 82974fbbf9e39910247ab9202e4b0990ba494a30

comment:16 Changed 6 years ago by Russell Taylor

Re #9341. Don't add the OpenMP flag on Windows.

It causes a warning in link.exe (that states it's not using the flag because it doesn't know what it means).

Changeset: 44c1ca48968171b7e2d9f29bbecdeeb54975205a

comment:17 Changed 6 years ago by Russell Taylor

  • Status changed from inprogress to verify
  • Resolution set to fixed

That's all the warnings that are in master. Others that remain in the Jenkins jobs are only in develop (i.e. open tickets) and the owners of the feature have been asked to get them sorted.

comment:18 Changed 6 years ago by Peter Peterson

  • Status changed from verify to verifying
  • Tester set to Peter Peterson

comment:19 Changed 6 years ago by Pete Peterson

  • Status changed from verifying to closed

Merge remote-tracking branch 'origin/bugfix/9341_doxygen_and_ccpcheck_warnings'

Full changeset: 48ed6441daad276b52008507cbb0e1d20a9ad928

comment:20 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 10184

Note: See TracTickets for help on using tickets.