Title: [228953] trunk/Source/WTF
Revision
228953
Author
commit-qu...@webkit.org
Date
2018-02-23 10:37:35 -0800 (Fri, 23 Feb 2018)

Log Message

warning: unused variable 'InitialBufferSize' in Assertions.cpp
https://bugs.webkit.org/show_bug.cgi?id=183076

Patch by Fujii Hironori <hironori.fu...@sony.com> on 2018-02-23
Reviewed by Yusuke Suzuki.

* wtf/Assertions.cpp: Moved the definition of InitialBufferSize to inside #if.

Modified Paths

Diff

Modified: trunk/Source/WTF/ChangeLog (228952 => 228953)


--- trunk/Source/WTF/ChangeLog	2018-02-23 18:35:56 UTC (rev 228952)
+++ trunk/Source/WTF/ChangeLog	2018-02-23 18:37:35 UTC (rev 228953)
@@ -1,3 +1,12 @@
+2018-02-23  Fujii Hironori  <hironori.fu...@sony.com>
+
+        warning: unused variable 'InitialBufferSize' in Assertions.cpp
+        https://bugs.webkit.org/show_bug.cgi?id=183076
+
+        Reviewed by Yusuke Suzuki.
+
+        * wtf/Assertions.cpp: Moved the definition of InitialBufferSize to inside #if.
+
 2018-02-22  Yusuke Suzuki  <utatane....@gmail.com>
 
         Remove currentTime() / currentTimeMS()

Modified: trunk/Source/WTF/wtf/Assertions.cpp (228952 => 228953)


--- trunk/Source/WTF/wtf/Assertions.cpp	2018-02-23 18:35:56 UTC (rev 228952)
+++ trunk/Source/WTF/wtf/Assertions.cpp	2018-02-23 18:37:35 UTC (rev 228953)
@@ -87,8 +87,6 @@
     fputs(buffer, stderr);
 }
 
-static const constexpr unsigned InitialBufferSize { 256 };
-
 WTF_ATTRIBUTE_PRINTF(1, 0)
 static void vprintf_stderr_common(const char* format, va_list args)
 {
@@ -105,6 +103,7 @@
 #pragma clang diagnostic pop
 #endif
         CFIndex length = CFStringGetMaximumSizeForEncoding(CFStringGetLength(str.get()), kCFStringEncodingUTF8);
+        constexpr unsigned InitialBufferSize { 256 };
         Vector<char, InitialBufferSize> buffer(length + 1);
 
         CFStringGetCString(str.get(), buffer.data(), length, kCFStringEncodingUTF8);
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to