Added test and applied fix for issue #67.

glynos · glynos · commit fdfeeb8aac95 · 2011-11-25T19:47:35.000+01:00
diff --git a/boost/network/uri/uri.hpp b/boost/network/uri/uri.hpp
@@ -72,8 +72,9 @@ class uri
         return *this;
     }
 
-    uri &operator = (const string_type &uri_) {
-        uri(uri_).swap(*this);
+    uri &operator = (const string_type &uri) {
+        uri_ = uri;
+        parse();
         return *this;
     }
 
diff --git a/libs/network/test/uri/url_test.cpp b/libs/network/test/uri/url_test.cpp
@@ -198,3 +198,13 @@ BOOST_AUTO_TEST_CASE(range_test)
     BOOST_REQUIRE(uri::valid(instance));
     BOOST_CHECK(boost::equal(instance, url));
 }
+
+BOOST_AUTO_TEST_CASE(issue_67_test)
+{
+    const std::string site_name("http://www.google.com");
+    uri::uri bar0;
+    uri::uri bar1 = site_name;
+    bar0 = site_name;
+    BOOST_CHECK(uri::is_valid(bar0));
+    BOOST_CHECK(uri::is_valid(bar1));
+}

Original file line number	Diff line number	Diff line change
`@@ -72,8 +72,9 @@ class uri`
`72`	`72`	`return *this;`
`73`	`73`	`}`
`74`	`74`
`75`		`- uri &operator = (const string_type &uri_) {`
`76`		`- uri(uri_).swap(*this);`
	`75`	`+ uri &operator = (const string_type &uri) {`
	`76`	`+ uri_ = uri;`
	`77`	`+ parse();`
`77`	`78`	`return *this;`
`78`	`79`	`}`
`79`	`80`