doxygen: Fix member comments #14331

pull MarcoFalke wants to merge 1 commits into bitcoin:master from MarcoFalke:Mf1809-doxygenMembers changing 6 files +32 −32

MarcoFalke commented at 7:53 PM on September 26, 2018: member
Trailing comments must be indicted with the caret //!<.

Not all places do this right now, see for example https://dev.visucore.com/bitcoin/doxygen/txmempool_8h.html#a2bc6653552b5871101b6cbefdbaf251f, but they can be fixed with an almost-scripted-diff:
```
sed -i --regexp-extended -e 's/((,|;) *\/\/!) /\1< /g' $(git grep --extended-regexp -l  '(,|;)\s*//!\s')
```
(Same as [doxygen] Fix member comments #7793)
doxygen: Fix member comments fa69ac7614
MarcoFalke added the label Docs on Sep 26, 2018
DrahtBot commented at 10:27 PM on September 26, 2018: contributor
Reviewers, this pull request conflicts with the following ones:
- #11413 ([wallet] [rpc] sendtoaddress/sendmany: Add explicit feerate option by kallewoof)
If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.
DrahtBot cross-referenced this on Sep 27, 2018 from issue [wallet] [rpc] sendtoaddress/sendmany: Add explicit feerate option by kallewoof
jamesob commented at 2:10 PM on September 27, 2018: member

utACK https://github.com/bitcoin/bitcoin/pull/14331/commits/fa69ac761441af3e1195fbb4018b18233a4433d2
l2a5b1 commented at 4:32 PM on September 27, 2018: contributor

utACK fa69ac7
Empact commented at 6:13 PM on September 27, 2018: member

utACK fa69ac7
DrahtBot commented at 2:38 AM on September 28, 2018: contributor



Coverage Change (pull 14331) Reference (master)

Lines -0.0309 % 87.0361 %

Functions +0.0926 % 84.1130 %

Branches -0.0445 % 51.5451 %
promag commented at 9:46 AM on September 29, 2018: member

utACK fa69ac7.
MarcoFalke referenced this in commit 011c39c296 on Oct 1, 2018
MarcoFalke merged this on Oct 1, 2018
MarcoFalke closed this on Oct 1, 2018
MarcoFalke deleted the branch on Oct 1, 2018
Bushstar cross-referenced this on Oct 11, 2018 from issue Updates from bitcoin/master by Bushstar
hans-schmidt cross-referenced this on Jul 28, 2019 from issue Fix some formatting problems in Doxygen docs by hans-schmidt
jasonbcox referenced this in commit 44a1e1cd6c on Oct 11, 2019
PastaPastaPasta referenced this in commit 91d8dc6d17 on Aug 16, 2021
PastaPastaPasta referenced this in commit 569e449a4f on Aug 16, 2021
PastaPastaPasta referenced this in commit e3c2fddf96 on Aug 18, 2021
bitcoin locked this on Sep 8, 2021

Coverage	Change (pull 14331)	Reference (master)
Lines	-0.0309 %	87.0361 %
Functions	+0.0926 %	84.1130 %
Branches	-0.0445 %	51.5451 %

Contributors

Labels

Linked (view graph)

#7793 [doxygen] Fix member comments #34967 doc: Discourage trailing doxygen comments, and fix the broken ones