Fixed a typo and Added C++ syntax highlighting in the documentation #4857

pull himynameismartin wants to merge 2 commits into bitcoin:master from himynameismartin:master changing 2 files +7 −3

himynameismartin commented at 6:35 PM on September 5, 2014: contributor

No description provided.
Added C++ syntax highlighting 22a1cb188c
Fixed a typo c0c115451b
BitcoinPullTester commented at 6:49 PM on September 5, 2014: none

Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/p4857_c0c115451bad24571303766f2fe6a6d44815b1c6/ for binaries and test log. This test script verifies pulls every time they are updated. It, however, dies sometimes and fails to test properly. If you are waiting on a test, please check timestamps to verify that the test.log is moving at http://jenkins.bluematt.me/pull-tester/current/ Contact BlueMatt on freenode if something looks broken.
sipa commented at 8:18 PM on September 5, 2014: member

I think we should just remove this file. It's essentially an outdated design document for something nobody is currently working on.
himynameismartin commented at 9:27 PM on September 5, 2014: contributor

I was just reading the documentation for fun and found it a bit distracting so I've fixed it. I'll not be making any further changes, so we can just close this pull request and I'll leave the rest up to you?
laanwj commented at 12:26 PM on September 10, 2014: member

Well it isn't really outdated. The architecture is still as described in that document. Although a better place to have it would be as part of the class documentation (comments) in Doxygen, not a separate document.
laanwj commented at 6:59 AM on September 11, 2014: member

But I agree on closing this pull.
laanwj closed this on Sep 11, 2014
MarcoFalke locked this on Sep 8, 2021

Contributors