More tweaks

Author: kunitoki

modules/yup_core/network/yup_URL.cpp

@@ -265,6 +265,36 @@ URL URL::createWithoutParsing (const String& u)
     return URL (u, 0);
 }
 
+URL& URL::operator= (const URL& other)
+{
+    if (this != std::addressof (other))
+    {
+        url = other.url;
+        postData = other.postData;
+        parameterNames = other.parameterNames;
+        parameterValues = other.parameterValues;
+        anchor = other.anchor;
+        filesToUpload = other.filesToUpload;
+    }
+
+    return *this;
+}
+
+URL& URL::operator= (URL&& other)
+{
+    if (this != std::addressof (other))
+    {
+        url = std::move (other.url);
+        postData = std::move (other.postData);
+        parameterNames = std::move (other.parameterNames);
+        parameterValues = std::move (other.parameterValues);
+        anchor = std::move (other.anchor);
+        filesToUpload = std::move (other.filesToUpload);
+    }
+
+    return *this;
+}
+
 bool URL::operator== (const URL& other) const
 {
     return url == other.url

modules/yup_core/network/yup_URL.h

@@ -75,6 +75,12 @@ class YUP_API URL
     /** Move construct an URL. */
     URL (URL&& other);
 
+    /** Copy assign an URL. */
+    URL& operator= (const URL& other);
+
+    /** Move assign an URL. */
+    URL& operator= (URL&& other);
+
     /** Compares two URLs.
 
         All aspects of the URLs must be identical for them to match, including any parameters,

tests/yup_core/yup_URL.cpp

@@ -57,6 +57,46 @@ TEST_F (URLTests, DefaultConstruction)
     EXPECT_FALSE (url.isWellFormed());
 }
 
+TEST_F (URLTests, CopyConstruction)
+{
+    URL url1 ("http://[email protected]?q=test&page=2#abc");
+
+    URL url2 { url1 };
+    EXPECT_EQ (url1, url2);
+    EXPECT_EQ (url1.toString (true), url2.toString (true));
+    EXPECT_EQ (url1.toString (false), url2.toString (false));
+}
+
+TEST_F (URLTests, CopyAssignment)
+{
+    URL url1 ("http://[email protected]?q=test&page=2#abc");
+
+    URL url2;
+    url2 = url1;
+    EXPECT_EQ (url1, url2);
+    EXPECT_EQ (url1.toString (true), url2.toString (true));
+    EXPECT_EQ (url1.toString (false), url2.toString (false));
+}
+
+TEST_F (URLTests, MoveConstruction)
+{
+    auto urlString = String ("http://[email protected]?q=test&page=2#abc");
+    URL url1 { urlString };
+
+    URL url2 { std::move (url1) };
+    EXPECT_EQ (url2.toString (true), urlString);
+}
+
+TEST_F (URLTests, MoveAssignment)
+{
+    auto urlString = String ("http://[email protected]?q=test&page=2#abc");
+    URL url1 { urlString };
+
+    URL url2;
+    url2 = std::move (url1);
+    EXPECT_EQ (url2.toString (true), urlString);
+}
+
 TEST_F (URLTests, StringConstruction)
 {
     // Basic URL