Skip to content

Commit

Permalink
Removed the tests that contain std::tuple with non default constructi…
Browse files Browse the repository at this point in the history
…ble class on MSVC2015.

It seems that MSVC2015 requires tuple elements' default constructor during 'as' process.
I don't know why.
  • Loading branch information
redboltz committed Aug 15, 2015
1 parent 0fafa7a commit 1c810bc
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 0 deletions.
6 changes: 6 additions & 0 deletions test/boost_fusion.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -140,6 +140,11 @@ BOOST_FUSION_ADAPT_STRUCT(
f3 // *3
)

#if !defined(_MSC_VER)

// boost::fusion's adaptor uses std::tuple.
// MSVC2015's std::tuple requires default constructor during 'as' process.
// I don't know why...
TEST(MSGPACK_BOOST, pack_convert_no_def_con)
{
std::stringstream ss;
Expand All @@ -151,6 +156,7 @@ TEST(MSGPACK_BOOST, pack_convert_no_def_con)
EXPECT_TRUE(val1 == val2);
}

#endif // !defined(_MSC_VER)

#endif // !defined(MSGPACK_USE_CPP03

Expand Down
6 changes: 6 additions & 0 deletions test/msgpack_cpp11.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -456,6 +456,10 @@ TEST(MSGPACK_NO_DEF_CON_PAIR, simple_buffer)
EXPECT_EQ(val1, val2);
}

#if !defined(_MSC_VER)

// MSVC2015's std::tuple requires default constructor during 'as' process.
// I don't know why...
TEST(MSGPACK_NO_DEF_CON_TUPLE, simple_buffer)
{
std::tuple<no_def_con, no_def_con, no_def_con_composite> val1 {1, 2, 3};
Expand All @@ -480,6 +484,8 @@ TEST(MSGPACK_NO_DEF_CON_MSGPACK_TUPLE, simple_buffer)
EXPECT_EQ(val1, val2);
}

#endif // !define(_MSC_VER)

TEST(MSGPACK_NO_DEF_FORWARD_LIST, simple_buffer)
{
std::forward_list<no_def_con> val1 { 1, 2, 3 };
Expand Down

0 comments on commit 1c810bc

Please sign in to comment.