Log Message
Fix -Wredundant-move warnings https://bugs.webkit.org/show_bug.cgi?id=230282
Patch by Michael Catanzaro <[email protected]> on 2021-09-14 Reviewed by Alex Christensen. * Platform/IPC/ArgumentCoders.cpp: (IPC::ArgumentCoder<CString>::decode): * Shared/soup/WebCoreArgumentCodersSoup.cpp: (IPC::ArgumentCoder<CertificateInfo>::decode):
Modified Paths
Diff
Modified: trunk/Source/WebKit/ChangeLog (282420 => 282421)
--- trunk/Source/WebKit/ChangeLog 2021-09-14 22:57:53 UTC (rev 282420)
+++ trunk/Source/WebKit/ChangeLog 2021-09-14 23:47:54 UTC (rev 282421)
@@ -1,3 +1,15 @@
+2021-09-14 Michael Catanzaro <[email protected]>
+
+ Fix -Wredundant-move warnings
+ https://bugs.webkit.org/show_bug.cgi?id=230282
+
+ Reviewed by Alex Christensen.
+
+ * Platform/IPC/ArgumentCoders.cpp:
+ (IPC::ArgumentCoder<CString>::decode):
+ * Shared/soup/WebCoreArgumentCodersSoup.cpp:
+ (IPC::ArgumentCoder<CertificateInfo>::decode):
+
2021-09-14 Wenson Hsieh <[email protected]>
[macOS] Avoid computing post-layout EditorState data unless necessary
Modified: trunk/Source/WebKit/Platform/IPC/ArgumentCoders.cpp (282420 => 282421)
--- trunk/Source/WebKit/Platform/IPC/ArgumentCoders.cpp 2021-09-14 22:57:53 UTC (rev 282420)
+++ trunk/Source/WebKit/Platform/IPC/ArgumentCoders.cpp 2021-09-14 23:47:54 UTC (rev 282421)
@@ -122,7 +122,7 @@
if (!decoder.decodeFixedLengthData(reinterpret_cast<uint8_t*>(buffer), *length, 1))
return std::nullopt;
- return WTFMove(string);
+ return string;
}
template
std::optional<CString> ArgumentCoder<CString>::decode<Decoder>(Decoder&);
Modified: trunk/Source/WebKit/Shared/soup/WebCoreArgumentCodersSoup.cpp (282420 => 282421)
--- trunk/Source/WebKit/Shared/soup/WebCoreArgumentCodersSoup.cpp 2021-09-14 22:57:53 UTC (rev 282420)
+++ trunk/Source/WebKit/Shared/soup/WebCoreArgumentCodersSoup.cpp 2021-09-14 23:47:54 UTC (rev 282421)
@@ -88,7 +88,7 @@
certificateInfo.setTLSErrors(static_cast<GTlsCertificateFlags>(*tlsErrors));
}
- return WTFMove(certificateInfo);
+ return certificateInfo;
}
template std::optional<CertificateInfo> ArgumentCoder<CertificateInfo>::decode<Decoder>(Decoder&);
template std::optional<CertificateInfo> ArgumentCoder<CertificateInfo>::decode<WebKit::PCM::Decoder>(WebKit::PCM::Decoder&);
_______________________________________________ webkit-changes mailing list [email protected] https://lists.webkit.org/mailman/listinfo/webkit-changes
