Skip to content
This repository was 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
Development

Successfully merging this pull request may close these issues.

3 participants