Skip to content

Fix up test warning in VS2017, build v5/secp256k1. #162

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

Merged
merged 2 commits into from
Dec 30, 2017
Merged

Fix up test warning in VS2017, build v5/secp256k1. #162

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
ed12d15
Build verison3 from version5/secp256k1.
evoskuil Dec 28, 2017
35c3315
Fix up test warning in VS2017.
evoskuil Dec 26, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion install.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -682,7 +682,7 @@ build_all()
{
build_from_tarball_boost $BOOST_URL $BOOST_ARCHIVE bzip2 . $PARALLEL "$BUILD_BOOST" "${BOOST_OPTIONS[@]}"
build_from_github zeromq libzmq master $PARALLEL ${ZMQ_OPTIONS[@]} "$@"
build_from_github libbitcoin secp256k1 version4 $PARALLEL ${SECP256K1_OPTIONS[@]} "$@"
build_from_github libbitcoin secp256k1 version5 $PARALLEL ${SECP256K1_OPTIONS[@]} "$@"
build_from_github libbitcoin libbitcoin version3 $PARALLEL ${BITCOIN_OPTIONS[@]} "$@"
build_from_travis libbitcoin libbitcoin-protocol version3 $PARALLEL ${BITCOIN_PROTOCOL_OPTIONS[@]} "$@"
}
Expand Down
6 changes: 3 additions & 3 deletions test/zmq/identifiers.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -55,16 +55,16 @@ BOOST_AUTO_TEST_CASE(identifiers__contains__default__not_zero)

BOOST_AUTO_TEST_CASE(identifiers__contains__one_value__expected)
{
const auto expected = 42u;
const intptr_t expected = 42u;
identifiers_fixture instance;
instance.push(reinterpret_cast<void*>(expected));
BOOST_REQUIRE(instance.contains(expected));
}

BOOST_AUTO_TEST_CASE(identifiers__contains__two_values__expected)
{
const auto expected1 = 99u;
const auto expected2 = 42u;
const intptr_t expected1 = 99u;
const intptr_t expected2 = 42u;
identifiers_fixture instance;
instance.push(reinterpret_cast<void*>(expected1));
instance.push(reinterpret_cast<void*>(expected2));
Expand Down