unordered_set and unordered_map serialization

[Jim Bell <jim@…>]

This patch provides serialization for std::tr1 containers unordered_set, unordered_multiset, unordered_map, and unordered_multimap.

Code using boost::unordered_... must instead use std::tr1. The interface is identical.

It includes boost TR1 header files for portability (<boost/tr1/unordered_set.hpp> or <boost/tr1/unordered_map.hpp>). See <​http://lists.boost.org/boost-users/2011/02/66535.php>

The patch file adapts the serialization tests test_map.cpp and test_set.cpp to include unordered variants. This patch requires ​changeset 69520 (see ​Ticket #5254).

This has been verified on MinGW-32/gcc-4.4 and MSVC-8. It should produce no spurious warnings on compile.

[Jim Bell <jim@…>]

boost/serialization/unordered_set.hpp

[Jim Bell <jim@…>]

boost/serialization/unordered_map.hpp

[Jim Bell <jim@…>]

boost/serialization/unordered_collections_load_imp.hpp

[Jim Bell <jim@…>]

boost/serialization/unordered_collections_save_imp.hpp

[Jim Bell <jim@…>]

Patch for libs/serialization/test (test_map.cpp and test_set.cpp)

[jsd@…]

'boost/serialization/detail/stack_constructor.hpp' seems to be missing from the includes in 'boost/serialization/unordered_collections_load_imp.hpp'

gcc version 4.5.1 20100924 (Red Hat 4.5.1-4) (GCC), fedora 14

[Jim Bell <jim@…>]

boost/serialization/unordered_collections_load_imp.hpp (v2 -- supercedes previous)

[Jim Bell <jim@…>]

jsd: thanks for the feedback.

I merged into unordered_collections_load_imp.hpp the same headers that collections_load_imp.hpp includes, boost/serialization/detail/stack_constructor.hpp being one. I checked unordered_collections_save_imp.hpp and it seemed ok.

Passes all tests against MinGW-gcc-4.4 and MSVC-8.

[anonymous]

what about boost own containers? how can they provide serialization for stl containers and not for boost::own containers, damn

[Robert Ramey]

I've added this to the library. I made changes to be compatible with c++11 rather than TR1. Will show up in the development branch shortly.

Robert Ramey

[anonymous]

Replying to ramey:

I've added this to the library. I made changes to be compatible with c++11 rather than TR1. Will show up in the development branch shortly.

Robert Ramey

How shortly is "shortly"? I can't find it yet and it's at least 3 years now.

Thanks for a beautiful library

[maxim.yegorushkin@…]

I just hit this issue today, when these patches will be integrated into boost please?

[querier]

Could this please be merged? I'm copying the file for local use, but I'd really like this to be distributed with boost.

[Robert Ramey]

my records show:

commit to develop commit 2140063

include/boost/serialization/stack.hpp include/boost/serialization/unordered_collections_load_imp.hpp include/boost/serialization/unordered_collections_save_imp.hpp include/boost/serialization/unordered_map.hpp include/boost/serialization/unordered_set.hpp include/boost/serialization/void_cast.hpp test/Jamfile.v2 test/test_array.cpp test/test_deque.cpp test/test_list.cpp test/test_map.cpp test/test_pimpl.cpp test/test_priority_queue.cpp test/test_private_base.cpp test/test_queue.cpp test/test_set.cpp test/test_stack.cpp test/test_vector.cpp

and merge to master commit 484874f

....

Does this not address this ticket?