i2p doc changes #255

Closed
opened 1 year ago by duke · 5 comments
duke commented 1 year ago
Owner
https://github.com/bitcoin/bitcoin/commit/0ed9cc58928d98633fb59922ebacbcf248d667cb https://github.com/bitcoin/bitcoin/commit/dffa31945729d110e4fff0fa092c077b5ec9e0eb https://github.com/bitcoin/bitcoin/pull/26838/files
duke added the
documentation
label 1 year ago
duke added the
i2p
label 1 year ago
Collaborator

bitcoin's src/i2p.h and src/i2p.cpp have new code in them, so is this just a documentation change?

bitcoin's `src/i2p.h` and `src/i2p.cpp` have new code in them, so is this just a documentation change?
Poster
Owner

@jahway603 I don't think I understand your question. Where are you seeing changes to i2p.h or i2p.cpp ?

@jahway603 I don't think I understand your question. Where are you seeing changes to i2p.h or i2p.cpp ?
Collaborator
I noticed the code differences when looking at how `doc/i2p.md` needed to change i2p.cpp https://github.com/bitcoin/bitcoin/commits/master/src/i2p.cpp vs https://git.hush.is/hush/hush3/commits/branch/master/src/i2p.cpp i2p.h https://github.com/bitcoin/bitcoin/commits/master/src/i2p.h vs https://git.hush.is/hush/hush3/commits/branch/master/src/i2p.h
Poster
Owner

@jahway603 this issue is about porting the documentation improvements from the pull request linked in the description. There are no functional changes, the changes to the code are just refactoring. The changes you mention to i2p.{cpp,h} are on the latest master branch of BTC, which may have additional changes. If you want to look at the difference between a file in our codebase versus in BTC I suggest running colordiff -u on the two files.

@jahway603 this issue is about porting the documentation improvements from the pull request linked in the description. There are no functional changes, the changes to the code are just refactoring. The changes you mention to i2p.{cpp,h} are on the latest master branch of BTC, which may have additional changes. If you want to look at the difference between a file in our codebase versus in BTC I suggest running `colordiff -u` on the two files.
Collaborator

Closing as PR #359 resolved this issue

Closing as PR https://git.hush.is/hush/hush3/pulls/359 resolved this issue
jahway603 closed this issue 6 months ago
Sign in to join this conversation.
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.