Fixed bug issue 111. It seems there was a problem in the assignment o…

…perators where the parsing was not done correctly, causing undefined behaviour.
kvikas · Jun 30, 2012 · bc2fbde · bc2fbde
1 parent 51fe256
commit bc2fbde
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 9 deletions.
diff --git a/include/network/uri/uri.hpp b/include/network/uri/uri.hpp
@@ -45,11 +45,6 @@ class BOOST_URI_DECL uri {
 
     }
 
-    //uri(const value_type *uri)
-    //    : uri_(uri), is_valid_(false) {
-    //    parse();
-    //}
-
     uri(const string_type &uri)
         : uri_(uri), is_valid_(false) {
         parse();
@@ -69,13 +64,15 @@ class BOOST_URI_DECL uri {
     }
 
     uri &operator = (const uri &other) {
-        uri(other).swap(*this);
+		uri_ = other.uri_;
+		parse();
         return *this;
     }
 
     uri &operator = (const string_type &uri_string) {
-		uri(uri_string).swap(*this);
-        return *this;
+		uri_ = uri_string;
+		parse();
+		return *this;
     }
 
     ~uri() {

diff --git a/libs/network/example/CMakeLists.txt b/libs/network/example/CMakeLists.txt
@@ -121,8 +121,8 @@ target_link_libraries(hello_world_client
     cppnetlib-http-client
     cppnetlib-http-client-connections)
 
-target_link_libraries(uri_builder ${OPENSSL_LIBRARIES})
 if (OPENSSL_FOUND)
+  target_link_libraries(uri_builder ${OPENSSL_LIBRARIES})
   #target_link_libraries(http_client ${OPENSSL_LIBRARIES})
   target_link_libraries(simple_wget ${OPENSSL_LIBRARIES})
   target_link_libraries(atom_reader ${OPENSSL_LIBRARIES})