apply doxygen format to sync.h and change license #5502

pull Diapolo wants to merge 1 commits into bitcoin:master from Diapolo:format_sync changing 1 files +40 −45

Diapolo commented at 5:50 PM on December 17, 2014: none

No description provided.
jonasschnelli commented at 9:44 AM on December 18, 2014: contributor

ACK
fanquake commented at 9:54 AM on December 19, 2014: member

The reason I'd left that undoxified was because the big comment seems to make more sense just in the file. This is how it appears in the doxygen output.

ACK changing the other lines, but I think 16 - 48 should be left as is.
apply doxygen format to sync.h and change license 0ca4ea7b69
Diapolo commented at 12:56 PM on December 26, 2014: none

@laanwj What's your oppinion here?
jonasschnelli commented at 10:23 AM on December 29, 2014: contributor

@Diapolo i agree with @fanquake The big ASCII-Visual-//// blocks should be not within a doxygen block.
laanwj commented at 12:57 PM on December 29, 2014: member

if you want to include it in the doxygen documentation then it needs serious reformatting to make it readable. Otherwise I'd indeed suggest leaving it as-is.
jgarzik commented at 12:53 PM on December 31, 2014: contributor

NAK for reasons @laanwj indicated
jgarzik closed this on Dec 31, 2014
Diapolo deleted the branch on Jan 30, 2015
DrahtBot locked this on Sep 8, 2021

Contributors