Skip to content
This repository has been archived by the owner on Jun 17, 2022. It is now read-only.

Backporting changes from bitcoin #6

Closed
wants to merge 1 commit into from
Closed

Backporting changes from bitcoin #6

wants to merge 1 commit into from

Conversation

karelbilek
Copy link

bitcoin/bitcoin@64fb0ac

Original comment:

Declare single-argument (non-converting) constructors "explicit"
In order to avoid unintended implicit conversions.

bitcoin/bitcoin@64fb0ac

Original comment:

Declare single-argument (non-converting) constructors "explicit"
In order to avoid unintended implicit conversions.
@sipa
Copy link

sipa commented Sep 25, 2017

ACK 05584fb

Copy link

@gmaxwell gmaxwell left a comment

Choose a reason for hiding this comment

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

From the PR in Bitcoin: "For a more thorough discussion, see "C.46: By default, declare single-argument constructors explicit" in the C++ Core Guidelines (Stroustrup & Sutter)."

ACK.

@laanwj laanwj closed this in #7 Oct 11, 2017
laanwj added a commit that referenced this pull request Oct 11, 2017
…icit"

fe805ea Declare single-argument (non-converting) constructors "explicit" (practicalswift)

Pull request description:

  Rebased on #4.

  Closes #6, which stripped the original author.

Tree-SHA512: 4f4164fb1bbce54e5e28a5cf1327c4d7a2058e74384d1173de843acbfb71a3dc4a11e559650b86c6ac9063ea9c76c96fa371bff985d794c8dc904448f4ecd095
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
3 participants