Skip to content

Conversation

@benma
Copy link

@benma benma commented May 27, 2017

Start of a series of changes to clean up the instantiation of connman
by decoupling the command line arguments.

@benma
Copy link
Author

benma commented May 27, 2017

Pinging @theuni

@fanquake fanquake added the P2P label May 27, 2017
@benma
Copy link
Author

benma commented May 29, 2017

@fanquake I don't think the P2P label is accurate. It should be Refactoring, like #9897, which this PR is a subset of.

@fanquake fanquake added Refactoring and removed P2P labels May 30, 2017
src/net.cpp Outdated
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you change this to not introduce new Boost code.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

src/init.cpp Outdated
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Even though your only moving this, if we're going to touch these lines, can you remove Boost.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

Start of a series of changes to clean up the instantiation of connman
by decoupling the command line arguments.
@benma benma force-pushed the connmannoptions_seednodes branch from dee1831 to 5d67526 Compare May 30, 2017 07:22
Copy link
Member

@theuni theuni left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice. Ridding CConnman of the argument parsing has been on my todo for a long time.

utACK 5d67526

@laanwj laanwj merged commit 5d67526 into bitcoin:master May 31, 2017
laanwj added a commit that referenced this pull request May 31, 2017
5d67526 add SeedNodes to CConnman::Options (Marko Bencun)

Tree-SHA512: d732ab08697419f896d9598fa3c30030ddbe3b82a99a398d813ebcd5e1fc610ad1043fe0f4c51a7ec5c4d70ce93a9ac5a5c7a91fcaae046ec4f1f7c41523a360
@benma benma deleted the connmannoptions_seednodes branch June 1, 2017 07:19
PastaPastaPasta pushed a commit to PastaPastaPasta/dash that referenced this pull request Jun 24, 2019
5d67526 add SeedNodes to CConnman::Options (Marko Bencun)

Tree-SHA512: d732ab08697419f896d9598fa3c30030ddbe3b82a99a398d813ebcd5e1fc610ad1043fe0f4c51a7ec5c4d70ce93a9ac5a5c7a91fcaae046ec4f1f7c41523a360
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Sep 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants