Libraries: Use default constructors/destructors in LibHTTP

https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#cother-other-default-operation-rules

"The compiler is more likely to get the default semantics right and
you cannot implement these functions better than the compiler."

Author: ldm5180

Userland/Libraries/LibHTTP/HttpRequest.cpp

@@ -1,5 +1,6 @@
 /*
  * Copyright (c) 2018-2020, Andreas Kling <kling@serenityos.org>
+ * Copyright (c) 2022, the SerenityOS developers.
  *
  * SPDX-License-Identifier: BSD-2-Clause
  */
@@ -11,14 +12,6 @@
 
 namespace HTTP {
 
-HttpRequest::HttpRequest()
-{
-}
-
-HttpRequest::~HttpRequest()
-{
-}
-
 String HttpRequest::method_name() const
 {
     switch (m_method) {

Userland/Libraries/LibHTTP/HttpRequest.h

@@ -1,5 +1,6 @@
 /*
  * Copyright (c) 2018-2020, Andreas Kling <kling@serenityos.org>
+ * Copyright (c) 2022, the SerenityOS developers.
  *
  * SPDX-License-Identifier: BSD-2-Clause
  */
@@ -34,8 +35,8 @@ class HttpRequest {
         String password;
     };
 
-    HttpRequest();
-    ~HttpRequest();
+    HttpRequest() = default;
+    ~HttpRequest() = default;
 
     String const& resource() const { return m_resource; }
     Vector<Header> const& headers() const { return m_headers; }

Userland/Libraries/LibHTTP/HttpResponse.cpp

@@ -1,5 +1,6 @@
 /*
  * Copyright (c) 2018-2020, Andreas Kling <kling@serenityos.org>
+ * Copyright (c) 2022, the SerenityOS developers.
  *
  * SPDX-License-Identifier: BSD-2-Clause
  */
@@ -15,10 +16,6 @@ HttpResponse::HttpResponse(int code, HashMap<String, String, CaseInsensitiveStri
 {
 }
 
-HttpResponse::~HttpResponse()
-{
-}
-
 StringView HttpResponse::reason_phrase_for_code(int code)
 {
     VERIFY(code >= 100 && code <= 599);

Userland/Libraries/LibHTTP/HttpResponse.h

@@ -1,5 +1,6 @@
 /*
  * Copyright (c) 2018-2020, Andreas Kling <kling@serenityos.org>
+ * Copyright (c) 2022, the SerenityOS developers.
  *
  * SPDX-License-Identifier: BSD-2-Clause
  */
@@ -14,7 +15,7 @@ namespace HTTP {
 
 class HttpResponse : public Core::NetworkResponse {
 public:
-    virtual ~HttpResponse() override;
+    virtual ~HttpResponse() override = default;
     static NonnullRefPtr<HttpResponse> create(int code, HashMap<String, String, CaseInsensitiveStringTraits>&& headers, size_t downloaded_size)
     {
         return adopt_ref(*new HttpResponse(code, move(headers), downloaded_size));

Userland/Libraries/LibHTTP/Job.cpp

@@ -1,5 +1,6 @@
 /*
  * Copyright (c) 2018-2020, Andreas Kling <kling@serenityos.org>
+ * Copyright (c) 2022, the SerenityOS developers.
  *
  * SPDX-License-Identifier: BSD-2-Clause
  */
@@ -77,10 +78,6 @@ Job::Job(HttpRequest&& request, Core::Stream::Stream& output_stream)
 {
 }
 
-Job::~Job()
-{
-}
-
 void Job::start(Core::Stream::Socket& socket)
 {
     VERIFY(!m_socket);

Userland/Libraries/LibHTTP/Job.h

@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2020, the SerenityOS developers.
+ * Copyright (c) 2020-2022, the SerenityOS developers.
  *
  * SPDX-License-Identifier: BSD-2-Clause
  */
@@ -21,7 +21,7 @@ class Job : public Core::NetworkJob {
 
 public:
     explicit Job(HttpRequest&&, Core::Stream::Stream&);
-    virtual ~Job() override;
+    virtual ~Job() override = default;
 
     virtual void start(Core::Stream::Socket&) override;
     virtual void shutdown(ShutdownMode) override;