Skip to content
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

Encapsulate and lock stat_datapoint, fixes core_test tsan #1665

Merged

Conversation

Projects
2 participants
@cryptocode
Copy link
Collaborator

commented Jan 30, 2019

Also a clang-format guard where versions disagree.

@cryptocode cryptocode added this to the V18.0 milestone Jan 30, 2019

@cryptocode cryptocode self-assigned this Jan 30, 2019

@cryptocode cryptocode requested a review from SergiySW Jan 30, 2019

@cryptocode cryptocode added this to During RC in V18 Jan 30, 2019

@SergiySW

This comment has been minimized.

Copy link
Collaborator

commented Jan 30, 2019

Strange miniupnp changes

@cryptocode cryptocode force-pushed the cryptocode:stats/encapsulate-data_point branch from 7df3618 to 6f51a81 Jan 30, 2019

@SergiySW
Copy link
Collaborator

left a comment

LGTM

@cryptocode cryptocode merged commit 70532b5 into nanocurrency:master Jan 30, 2019

2 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@cryptocode cryptocode deleted the cryptocode:stats/encapsulate-data_point branch Jan 30, 2019

@zhyatt zhyatt moved this from RC2 to CP 3/RC 1 (2018-02-01) in V18 Feb 18, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.