Title: [262430] trunk/Source/WebCore
Revision
262430
Author
commit-qu...@webkit.org
Date
2020-06-02 11:10:55 -0700 (Tue, 02 Jun 2020)

Log Message

Pass nullptr for the 2nd argument of FileReaderLoader
https://bugs.webkit.org/show_bug.cgi?id=212642

Patch by Tetsuharu Ohzeki <tetsuharu.ohz...@gmail.com> on 2020-06-02
Reviewed by Darin Adler.

Instead of passing `0`, `nullptr` is better
because `FileReaderLoader` takes a pointer.

* fileapi/FileReaderSync.cpp:
(WebCore::FileReaderSync::readAsArrayBuffer):
(WebCore::FileReaderSync::readAsBinaryString):
(WebCore::FileReaderSync::readAsText):
(WebCore::FileReaderSync::readAsDataURL):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (262429 => 262430)


--- trunk/Source/WebCore/ChangeLog	2020-06-02 17:14:48 UTC (rev 262429)
+++ trunk/Source/WebCore/ChangeLog	2020-06-02 18:10:55 UTC (rev 262430)
@@ -1,3 +1,19 @@
+2020-06-02  Tetsuharu Ohzeki  <tetsuharu.ohz...@gmail.com>
+
+        Pass nullptr for the 2nd argument of FileReaderLoader 
+        https://bugs.webkit.org/show_bug.cgi?id=212642
+
+        Reviewed by Darin Adler.
+
+        Instead of passing `0`, `nullptr` is better
+        because `FileReaderLoader` takes a pointer.
+
+        * fileapi/FileReaderSync.cpp:
+        (WebCore::FileReaderSync::readAsArrayBuffer):
+        (WebCore::FileReaderSync::readAsBinaryString):
+        (WebCore::FileReaderSync::readAsText):
+        (WebCore::FileReaderSync::readAsDataURL):
+
 2020-06-02  Tim Horton  <timothy_hor...@apple.com>
 
         UIColor and NSColor WebCore::Color factories should return invalid colors for nil input colors

Modified: trunk/Source/WebCore/fileapi/FileReaderSync.cpp (262429 => 262430)


--- trunk/Source/WebCore/fileapi/FileReaderSync.cpp	2020-06-02 17:14:48 UTC (rev 262429)
+++ trunk/Source/WebCore/fileapi/FileReaderSync.cpp	2020-06-02 18:10:55 UTC (rev 262430)
@@ -45,7 +45,7 @@
 
 ExceptionOr<RefPtr<ArrayBuffer>> FileReaderSync::readAsArrayBuffer(ScriptExecutionContext& scriptExecutionContext, Blob& blob)
 {
-    FileReaderLoader loader(FileReaderLoader::ReadAsArrayBuffer, 0);
+    FileReaderLoader loader(FileReaderLoader::ReadAsArrayBuffer, nullptr);
     auto result = startLoading(scriptExecutionContext, loader, blob);
     if (result.hasException())
         return result.releaseException();
@@ -54,13 +54,13 @@
 
 ExceptionOr<String> FileReaderSync::readAsBinaryString(ScriptExecutionContext& scriptExecutionContext, Blob& blob)
 {
-    FileReaderLoader loader(FileReaderLoader::ReadAsBinaryString, 0);
+    FileReaderLoader loader(FileReaderLoader::ReadAsBinaryString, nullptr);
     return startLoadingString(scriptExecutionContext, loader, blob);
 }
 
 ExceptionOr<String> FileReaderSync::readAsText(ScriptExecutionContext& scriptExecutionContext, Blob& blob, const String& encoding)
 {
-    FileReaderLoader loader(FileReaderLoader::ReadAsText, 0);
+    FileReaderLoader loader(FileReaderLoader::ReadAsText, nullptr);
     loader.setEncoding(encoding);
     return startLoadingString(scriptExecutionContext, loader, blob);
 }
@@ -67,7 +67,7 @@
 
 ExceptionOr<String> FileReaderSync::readAsDataURL(ScriptExecutionContext& scriptExecutionContext, Blob& blob)
 {
-    FileReaderLoader loader(FileReaderLoader::ReadAsDataURL, 0);
+    FileReaderLoader loader(FileReaderLoader::ReadAsDataURL, nullptr);
     loader.setDataType(blob.type());
     return startLoadingString(scriptExecutionContext, loader, blob);
 }
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to