Changeset 249280 in webkit

Timestamp:

Aug 29, 2019 10:11:05 AM (5 years ago)

Author:

Chris Dumez

Message:

CompletionHandler default constructor does not initialize m_wasConstructedOnMainThread
​https://bugs.webkit.org/show_bug.cgi?id=201249

Reviewed by Joseph Pecoraro and Alex Christensen.

• wtf/CompletionHandler.h:

(WTF::CompletionHandler<Out):

Location:

trunk/Source/WTF

Files:

• ChangeLog (modified) (1 diff)
• wtf/CompletionHandler.h (modified) (2 diffs)

trunk/Source/WTF/ChangeLog

@@ +1 @@
+2019-08-29  Chris Dumez  <cdumez@apple.com>
+
+        CompletionHandler default constructor does not initialize `m_wasConstructedOnMainThread`
+        https://bugs.webkit.org/show_bug.cgi?id=201249
+
+        Reviewed by Joseph Pecoraro and Alex Christensen.
+
+        * wtf/CompletionHandler.h:
+        (WTF::CompletionHandler<Out):
+
 2019-08-29  Chris Dumez  <cdumez@apple.com>
 

trunk/Source/WTF/wtf/CompletionHandler.h

@@ -43 +43 @@
     CompletionHandler(CallableType&& callable)
         : m_function(WTFMove(callable))
-#if !ASSERT_DISABLED
-        , m_wasConstructedOnMainThread(isMainThread())
-#endif
     {
     }
@@ -69 +66 @@
     Function<Out(In...)> m_function;
 #if !ASSERT_DISABLED
-    bool m_wasConstructedOnMainThread;
+    bool m_wasConstructedOnMainThread { isMainThread() };
 #endif
 };