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.

Sign up for GitHub

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

Jump to bottom

Improves checksum error message #3031

Merged
merged 6 commits into from
Nov 17, 2017
Merged

Improves checksum error message #3031

merged 6 commits into from
Nov 17, 2017

Commits on Nov 17, 2017

  1. Improves address literal checksum error message

    @axic
    wadeAlexC authored and axic committed Nov 17, 2017
    Configuration menu
    Copy the full SHA
    8a6692b View commit details
    Browse the repository at this point in the history

  2. Ensure that non-hex characters are caught in address checksumming

    @axic
    axic committed Nov 17, 2017
    Configuration menu
    Copy the full SHA
    6ebc094 View commit details
    Browse the repository at this point in the history

  3. Do not try to display checksummed address for too-short/long address … 

    …literals
    @axic
    axic committed Nov 17, 2017
    Configuration menu
    Copy the full SHA
    1d5dd90 View commit details
    Browse the repository at this point in the history

  4. Show checksummed address always (prepend with 0)

    @axic
    axic committed Nov 17, 2017
    Configuration menu
    Copy the full SHA
    8d26894 View commit details
    Browse the repository at this point in the history

  5. Extend address checksum tests

    @axic
    axic committed Nov 17, 2017
    Configuration menu
    Copy the full SHA
    f7249ab View commit details
    Browse the repository at this point in the history

  6. Add tests for getChecksummedAddress

    @axic
    axic committed Nov 17, 2017
    Configuration menu
    Copy the full SHA
    53796c0 View commit details
    Browse the repository at this point in the history