Ticket #9341 (closed: fixed)
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: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
