-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Comment updates - InstantSend #2062
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
thephez
commented
May 3, 2018
- Adjust some comments so they will be picked up by doxygen
- Add a few brief descriptions
|
Might be nice to gradually update things so doxygen picks them up and also add additional detail on some of the more obscure stuff. Figured I would do something small like this and see what the feedback was... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good 👍
utACK
|
Note: will probably cause some minor merge conflicts for #2040 due to removal of |
|
Ah, explains my confusion. I thought those had been removed and was surprised to see them still in there. Did not think to check the open PRs. 🙄 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
utACK, Thanks for this! It's nice to get more comments/documentation on these parts of the code.
* [Comment] - Minor Instantx comment updates and additions * [Comment] - Update to doxygen recognised format - Capitalisation changes * [Comment] - Add some minor InstantSend comments
* [Comment] - Minor Instantx comment updates and additions * [Comment] - Update to doxygen recognised format - Capitalisation changes * [Comment] - Add some minor InstantSend comments