[uri] Added a test to check that the URI can be used as a key in a hash set.

Author: glynos

boost/network/uri/uri.hpp

@@ -18,6 +18,7 @@
 # include <boost/range/iterator_range.hpp>
 # include <boost/lexical_cast.hpp>
 # include <boost/optional.hpp>
+# include <boost/functional/hash_fwd.hpp>
 
 
 namespace boost {
@@ -300,6 +301,16 @@ void swap(uri &lhs, uri &rhs) {
     lhs.swap(rhs);
 }
 
+inline
+std::size_t hash_value(const uri &uri_)
+{
+    std::size_t seed = 0;
+    for (uri::const_iterator it = begin(uri_); it != end(uri_); ++it) {
+        hash_combine(seed, *it);
+    }
+    return seed;
+}
+
 inline
 bool operator == (const uri &lhs, const uri &rhs) {
     return boost::equal(lhs, rhs);
@@ -318,7 +329,6 @@ bool operator < (const uri &lhs, const uri &rhs) {
 } // namespace network
 } // namespace boost
 
-
 # include <boost/network/uri/accessors.hpp>
 # include <boost/network/uri/directives.hpp>
 # include <boost/network/uri/builder.hpp>

libs/network/test/uri/uri_test.cpp

@@ -12,6 +12,8 @@
 #include <boost/range/algorithm/equal.hpp>
 #include <boost/scoped_ptr.hpp>
 #include <map>
+#include <set>
+#include <boost/unordered_set.hpp>
 
 
 using namespace boost::network;
@@ -316,14 +318,13 @@ BOOST_AUTO_TEST_CASE(encoded_uri_test) {
 BOOST_AUTO_TEST_CASE(copy_constructor_test) {
     uri::uri instance("http://www.example.com/");
     uri::uri copy = instance;
-    BOOST_CHECK(instance == copy);
+    BOOST_CHECK_EQUAL(instance, copy);
 }
 
 BOOST_AUTO_TEST_CASE(assignment_test) {
     uri::uri instance("http://www.example.com/");
     uri::uri copy;
     copy = instance;
-    BOOST_CHECK_EQUAL(instance.string(), copy.string());
     BOOST_CHECK_EQUAL(instance, copy);
 }
 
@@ -463,3 +464,17 @@ BOOST_AUTO_TEST_CASE(issue_104_test) {
 	instance.reset();
 	BOOST_CHECK_EQUAL(uri::scheme(copy), "http");
 }
+
+BOOST_AUTO_TEST_CASE(uri_set_test) {
+    std::set<uri::uri> uri_set;
+    uri_set.insert(uri::uri("http://www.example.com/"));
+    BOOST_REQUIRE(!uri_set.empty());
+    BOOST_CHECK_EQUAL((*uri_set.begin()), uri::uri("http://www.example.com/"));
+}
+
+BOOST_AUTO_TEST_CASE(uri_unordered_set_test) {
+    boost::unordered_set<uri::uri> uri_set;
+    uri_set.insert(uri::uri("http://www.example.com/"));
+    BOOST_REQUIRE(!uri_set.empty());
+    BOOST_CHECK_EQUAL((*uri_set.begin()), uri::uri("http://www.example.com/"));
+}