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

Remove unnecessary cast #1730

Merged

Conversation

Projects
2 participants
@cryptocode
Copy link
Collaborator

commented Feb 15, 2019

Tiny cleanup while looking at #406
There are more of these, but taking those in a PR after #1729 gets merged to avoid known merge conflicts.

@cryptocode cryptocode self-assigned this Feb 15, 2019

@cryptocode cryptocode added this to the V19.0 milestone Feb 15, 2019

@cryptocode cryptocode added this to CP0 in V19 Feb 15, 2019

@cryptocode cryptocode requested a review from SergiySW Feb 15, 2019

@SergiySW
Copy link
Collaborator

left a comment

LGTM

@cryptocode cryptocode merged commit eceb942 into nanocurrency:master Feb 17, 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:cleanup/remove-unnecessary-cast-1 branch Feb 17, 2019

@cryptocode cryptocode modified the milestones: V19.0, V18.0 Feb 18, 2019

@zhyatt zhyatt added this to During RC in V18 Feb 18, 2019

argakiig added a commit that referenced this pull request Feb 18, 2019

@zhyatt zhyatt moved this from RC2 to RC3 in V18 Feb 18, 2019

@zhyatt zhyatt removed this from CP0 (2019-02-27) in V19 Mar 5, 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.