Title: [290420] trunk/Source/WTF
- Revision
- 290420
- Author
- carlo...@webkit.org
- Date
- 2022-02-24 02:22:01 -0800 (Thu, 24 Feb 2022)
Log Message
Unreviewed. Fix compile warnings when asserts are disabled after r290418
* wtf/Threading.h:
(WTF::WTF_ASSERTS_ACQUIRED_CAPABILITY):
* wtf/WorkQueue.h:
(WTF::WTF_ASSERTS_ACQUIRED_CAPABILITY):
Modified Paths
Diff
Modified: trunk/Source/WTF/ChangeLog (290419 => 290420)
--- trunk/Source/WTF/ChangeLog 2022-02-24 09:42:27 UTC (rev 290419)
+++ trunk/Source/WTF/ChangeLog 2022-02-24 10:22:01 UTC (rev 290420)
@@ -1,3 +1,12 @@
+2022-02-24 Carlos Garcia Campos <cgar...@igalia.com>
+
+ Unreviewed. Fix compile warnings when asserts are disabled after r290418
+
+ * wtf/Threading.h:
+ (WTF::WTF_ASSERTS_ACQUIRED_CAPABILITY):
+ * wtf/WorkQueue.h:
+ (WTF::WTF_ASSERTS_ACQUIRED_CAPABILITY):
+
2022-02-24 Kimmo Kinnunen <kkinnu...@apple.com>
Thread safety analysis to assert "code is run sequentially" is not useful when code is mainly run with WorkQueues
Modified: trunk/Source/WTF/wtf/Threading.h (290419 => 290420)
--- trunk/Source/WTF/wtf/Threading.h 2022-02-24 09:42:27 UTC (rev 290419)
+++ trunk/Source/WTF/wtf/Threading.h 2022-02-24 10:22:01 UTC (rev 290420)
@@ -430,7 +430,11 @@
inline void assertIsCurrent(const Thread& thread) WTF_ASSERTS_ACQUIRED_CAPABILITY(thread)
{
+#if ASSERT_ENABLED
ASSERT(&thread == &Thread::current());
+#else
+ UNUSED_PARAM(thread);
+#endif
}
} // namespace WTF
Modified: trunk/Source/WTF/wtf/WorkQueue.h (290419 => 290420)
--- trunk/Source/WTF/wtf/WorkQueue.h 2022-02-24 09:42:27 UTC (rev 290419)
+++ trunk/Source/WTF/wtf/WorkQueue.h 2022-02-24 10:22:01 UTC (rev 290420)
@@ -122,6 +122,8 @@
{
#if ASSERT_ENABLED
workQueue.assertIsCurrent();
+#else
+ UNUSED_PARAM(workQueue);
#endif
}
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes
